bdemers commented on PR #1103:
URL:
https://github.com/apache/directory-scimple/pull/1103#issuecomment-5840112512
Follow-ups before merge:
- [ ] Replace `api-all` (shaded uber-jar) with explicit LDAP API
dependencies (`api-ldap-model`, `api-ldap-client-api`,
`api-ldap-codec-standalone`). Mixing it with ApacheDS duplicates every LDAP API
class on the classpath.
- [ ] Pin all `org.apache.directory.api` modules to `${version.ldap-api}` in
`dependencyManagement`. ApacheDS 2.0.0.AM27 pulls in 2.1.5 transitively.
- [ ] Recheck the `api-ldap-schema-data` exclusion once the above is done,
since `api-ldap-client-api` depends on it.
- [ ] Validate the id is a UUID in `findByEntryUuid` (users and groups)
before searching. Random ids currently hit LDAP and log
`ERR_13246_INVALID_VALUE_PER_SYNTAX` with a stack trace.
- [ ] Bump `version.ldap-api` to 2.1.9 once released.
--
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 unsubscribe, e-mail: [email protected]
For queries about this service, please contact Infrastructure at:
[email protected]
---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]
