gnodet commented on code in PR #25622:
URL: https://github.com/apache/camel/pull/25622#discussion_r3842769626
##########
parent/pom.xml:
##########
@@ -3704,6 +3705,11 @@
<artifactId>commons-lang3</artifactId>
<version>${commons-lang3-version}</version>
</dependency>
+ <dependency>
+ <groupId>org.apache.directory.api</groupId>
+ <artifactId>api-ldap-client-api</artifactId>
+ <version>${directory-api-version}</version>
+ </dependency>
Review Comment:
⚠️ **Version skew risk** — This pins only `api-ldap-client-api` to 2.1.8,
but `camel-ldif` pulls in 10 other `org.apache.directory.api` artifacts at
2.1.5 (transitively from `apacheds-core-api:2.0.0.AM27`): `api-i18n`,
`api-asn1-api`, `api-ldap-extras-codec-api`, `api-ldap-codec-core`,
`api-asn1-ber`, `api-ldap-extras-aci`, `api-ldap-extras-util`,
`api-ldap-model`, `api-ldap-schema-data`, and `api-util`.
The 2.1.8 release includes significant internal changes (AntLR parser
replacement, ASN.1 decoder fixes) that may assume consistent versions across
all modules. CI is green which is encouraging, but tests may not exercise all
code paths affected by the version mismatch.
Consider pinning all `org.apache.directory.api` artifacts to 2.1.8 for
consistency — the existing `directory-api-version` property name already
implies broader version management.
--
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]
