Skip to content

(WIP) (PE-33645) Fix tests #16

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

Draft
wants to merge 2 commits into
base: main
Choose a base branch
from

Conversation

mwaggett
Copy link

@mwaggett mwaggett commented Aug 2, 2022

see individual commit messages for details

When running tests on Java 11, we were getting the following error:
```
Uncaught exception in test fixture
expected: nil
  actual: java.lang.NoClassDefFoundError: sun/reflect/Reflection
   at org.apache.directory.server.core.avltree.ArrayMarshaller.deserialize (ArrayMarshaller.java:153)
   [...]
```
This commit adds an explicit dependency on apacheds-core-avl 1.5.6, as suggested
by spring-projects/spring-security#6804 (version 1.5.7
caused some compilation errors), in order to support Java 11.
@mwaggett mwaggett added the WIP Work in progress... label Aug 2, 2022
@mwaggett
Copy link
Author

mwaggett commented Aug 2, 2022

The new keystore was generated from the existing keystore via the following command:
keytool -importkeystore -srckeystore dev-resources/keystore-local.jks -destkeystore dev-resources/keystore-local-new.jks -deststoretype pkcs12
Keystore password: password
Self-signed cert password: secret

See #9 for how the old keystore was generated.

@CLAassistant
Copy link

CLAassistant commented Apr 19, 2023

CLA assistant check
Thank you for your submission! We really appreciate it. Like many open source projects, we ask that you sign our Contributor License Agreement before we can accept your contribution.
You have signed the CLA already but the status is still pending? Let us recheck it.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
WIP Work in progress...
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants