Re: [PR] Make Pooled LDAP connections return to pool when closed [directory-ldap-api]

2024-02-21 Thread via GitHub
elecharny commented on PR #79: URL: https://github.com/apache/directory-ldap-api/pull/79#issuecomment-1958831866 @bdemers give me a bir of time to review the PR, I'll be busy the next two days. Those classes are not frequently used, but every thing that makes it easier o use is

Re: [PR] Make Pooled LDAP connections return to pool when closed [directory-ldap-api]

2024-02-21 Thread via GitHub
bdemers commented on code in PR #79: URL: https://github.com/apache/directory-ldap-api/pull/79#discussion_r1498524948 ## ldap/client/api/src/main/java/org/apache/directory/ldap/client/api/LdapConnectionPool.java: ## @@ -123,6 +123,10 @@ public LdapConnection getConnection()

[PR] Make Pooled LDAP connections return to pool when closed (v2) [directory-ldap-api]

2024-02-21 Thread via GitHub
bdemers opened a new pull request, #80: URL: https://github.com/apache/directory-ldap-api/pull/80 Fixes: DIRAPI-398 -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific comment. To

Re: [PR] Make Pooled LDAP connections return to pool when closed [directory-ldap-api]

2024-02-21 Thread via GitHub
bdemers commented on code in PR #79: URL: https://github.com/apache/directory-ldap-api/pull/79#discussion_r1498516680 ## ldap/client/api/src/main/java/org/apache/directory/ldap/client/api/LdapConnectionPool.java: ## @@ -123,6 +123,10 @@ public LdapConnection getConnection()

[PR] Make Pooled LDAP connections return to pool when closed [directory-ldap-api]

2024-02-21 Thread via GitHub
bdemers opened a new pull request, #79: URL: https://github.com/apache/directory-ldap-api/pull/79 Fixes: DIRAPI-398 -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific comment. To

[PR] Bump org.codehaus.mojo:exec-maven-plugin from 3.1.1 to 3.2.0 [directory-studio]

2024-02-21 Thread via GitHub
dependabot[bot] opened a new pull request, #71: URL: https://github.com/apache/directory-studio/pull/71 Bumps [org.codehaus.mojo:exec-maven-plugin](https://github.com/mojohaus/exec-maven-plugin) from 3.1.1 to 3.2.0. Release notes Sourced from

[jira] [Created] (DIRAPI-398) Release Pooled LDAP Connections when closed

2024-02-21 Thread Brian Demers (Jira)
Brian Demers created DIRAPI-398: --- Summary: Release Pooled LDAP Connections when closed Key: DIRAPI-398 URL: https://issues.apache.org/jira/browse/DIRAPI-398 Project: Directory Client API Issue

Re: [I] Enriching calls to scim repository [directory-scimple]

2024-02-21 Thread via GitHub
erant10 commented on issue #449: URL: https://github.com/apache/directory-scimple/issues/449#issuecomment-1957329012 No it was about Guice binding the ScimResourceContextProvider with its `Http` implementation... I tried doing that using some internal jaxrs binding module we have but that

Re: [I] Enriching calls to scim repository [directory-scimple]

2024-02-21 Thread via GitHub
bdemers commented on issue #449: URL: https://github.com/apache/directory-scimple/issues/449#issuecomment-1957313052 Thanks for the update! As an aside, where your annoying injection issues related to field injection of the RequestContext and/or the mixing and matching of JAX-RS context

Re: [I] Enriching calls to scim repository [directory-scimple]

2024-02-21 Thread via GitHub
erant10 commented on issue #449: URL: https://github.com/apache/directory-scimple/issues/449#issuecomment-1957303239 @bdemers Sorry totally forgot to respond. Basically did exactly what you said - the abstraction ScimResourceContext injected with RequestContext which has everything I

[PR] Bump org.apache.maven.plugins:maven-shade-plugin from 3.5.1 to 3.5.2 [directory-kerby]

2024-02-21 Thread via GitHub
dependabot[bot] opened a new pull request, #337: URL: https://github.com/apache/directory-kerby/pull/337 Bumps [org.apache.maven.plugins:maven-shade-plugin](https://github.com/apache/maven-shade-plugin) from 3.5.1 to 3.5.2. Commits