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

Update dependencies, especially jBCrypt. #20

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

brunchboy
Copy link

jBCrypt-0.4 fixes an integer overflow which occurs with very large
log_rounds values. In order to incorporate that fix, I had to switch to
a different Maven artifact which tracks more recent releases of
jBCrypt.

While updating that, I updated the other dependencies (apart from
Clojure itself). If you would like to update those separately, I can
certainly understand that. Alternately, you may want to update Clojure
as well, now that the final release of 1.8 is out.

jBCrypt-0.4 fixes an integer overflow which occurs with very large
log_rounds values. In order to incorporate that fix, I had to switch to
a different Maven artifact which tracks more recent releases of
jBCrypt.

While updating that, I updated the other dependencies (apart from
Clojure itself). If you would like to update those separately, I can
certainly understand that. Alternately, you may want to update Clojure
as well, now that the final release of 1.8 is out.
@brunchboy
Copy link
Author

Ah, sigh. Apparently the de.svenkubiak/jBCrypt artifact was compiled to Java 8 classes, making it incompatible with the Java 7 environment of this build, so this update may not be feasible. Perhaps we can persuade the maintainer of the jBCrypt artifact you started out using to update to jBCrypt version 0.4.

But for now, I will just use my fork, since I am working in a Java 8 environment.

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.

1 participant