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.

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

Several updates #61

Merged
merged 19 commits into from
Aug 15, 2021
Merged

Several updates #61

merged 19 commits into from
Aug 15, 2021

Conversation

norrisjeremy
Copy link
Contributor

@norrisjeremy norrisjeremy commented Aug 12, 2021

  1. Reduce number of algorithms that are runtime checked to improve baseline performance (came up in Authentication with public key fails with 0.1.62 #47).
  2. Add system properties to allow overriding the default crypto algorithms used (in response to Flag to enable "legacy" algos on initialization? #59).
  3. Switch from PubkeyAcceptedKeyTypes to PubkeyAcceptedAlgorithms to match recent changes in OpenSSH.
  4. Prefer ssh-ed25519 to match recent changes in OpenSSH.
  5. Allow configuration of DHGEX sizes and align the default sizes with recommendations from RFC-8270.
  6. Directly import JZlib since it appears unmaintained upstream.
  7. Add license files for OpenJAX & jBCrypt.
  8. Update dependencies to latest versions.
  9. Switch Github CI from Java 15 to Java 16 since that is current version of OpenJDK.
  10. Modularize jarfile.
  11. Mark a few classes package private that are not intended to be public.

@norrisjeremy
Copy link
Contributor Author

FYI, with the Github CI change from Java 15 to 16, the CI checks above probably will hang until you update the project to expect them from Java 16.

Also, when I was sweeping through dependency updates, I wasn't sure if the wagon-ssh-external extension should be updated to a newer version? The version specified in pom.xml appears to be quite old.

@mwiede
Copy link
Owner

mwiede commented Aug 13, 2021

@norrisjeremy can you please update the Readme of the frontpage with some information on system properties, so that users know, what to to do in order to configure old algorithms/old behavior? Thanks for your contribution!

@norrisjeremy
Copy link
Contributor Author

Yep, I'll update the Readme later today or over the weekend and push an update to this PR.

@norrisjeremy
Copy link
Contributor Author

Let me know if the changes to the Readme look ok, or if you would like additional changes.

@mwiede mwiede merged commit 5f2762c into mwiede:master Aug 15, 2021
@norrisjeremy norrisjeremy deleted the 20210812 branch August 16, 2021 13:07
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants