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.

Sign up for GitHub

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

Jump to bottom

Rework LDAP ApacheDS tests to work with JDK 11 #9416

Closed
jzheaux opened this issue Feb 8, 2021 · 0 comments
Closed

Rework LDAP ApacheDS tests to work with JDK 11 #9416

jzheaux opened this issue Feb 8, 2021 · 0 comments
Assignees
@jzheaux
Labels
in: ldap An issue in spring-security-ldap type: enhancement A general enhancement
Milestone
5.5.0-M3

Comments

@jzheaux
Copy link
Contributor

jzheaux commented Feb 8, 2021

ApacheDS is not compatible with JDK 11, and the reason is that one of its classes imports sun.reflect.Reflection in order to formulate one of its log messages.

In Spring Security's LDAP tests, the ApacheDS class can be overridden to remove this log statement and make it so that those tests can run under JDK 11.
@jzheaux jzheaux added in: ldap An issue in spring-security-ldap type: enhancement A general enhancement labels Feb 8, 2021
@jzheaux jzheaux self-assigned this Feb 8, 2021
@jzheaux jzheaux mentioned this issue Feb 8, 2021
Closed
7 tasks
jzheaux added a commit to jzheaux/spring-security that referenced this issue Feb 17, 2021
@jzheaux
Add Java 8 Polyfill for Apache DS tests
f60cdb4 
Closes spring-projectsgh-9416
@jzheaux jzheaux added this to the 5.5.0-M3 milestone Apr 8, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@jzheaux jzheaux

Labels
in: ldap An issue in spring-security-ldap type: enhancement A general enhancement
Projects
None yet
Milestone
5.5.0-M3
Development

No branches or pull requests
1 participant
@jzheaux