Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

[qxkr2RlZ] apoc.load.ldap cant find config value due to missing #3727

Merged
merged 2 commits into from
Aug 24, 2023

Conversation

vga91
Copy link
Collaborator

@vga91 vga91 commented Aug 22, 2023

  • Added support apoc.loadldap<KEY>.conf to apoc.loadldap.<KEY>.conf (with dot after loadldap)
  • Added tests with correct and wrong config key
  • Added log warn with fallback: apoc.loadldap<KEY>.conf and changed DbsmRule to TestDatabaseManagementServiceBuilder to check logs via FileUtils.getLogDirectory()

@vga91 vga91 force-pushed the 5.11-load-ldap-wrong-config branch from c7f2136 to 1f42722 Compare August 24, 2023 07:32
@conker84 conker84 merged commit a7ac660 into 5.11 Aug 24, 2023
@conker84 conker84 deleted the 5.11-load-ldap-wrong-config branch August 24, 2023 10:01
vga91 added a commit to vga91/neo4j-apoc-procedures that referenced this pull request Aug 31, 2023
…j-contrib#3727)

* [qxkr2RlZ] apoc.load.ldap cant find config value due to missing

* [qxkr2RlZ] added fallback with config key without dot
vga91 added a commit that referenced this pull request Aug 31, 2023
… (#3756)

* [qxkr2RlZ] apoc.load.ldap cant find config value due to missing

* [qxkr2RlZ] added fallback with config key without dot
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants