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

Switch to SecureRandom for encryption randomness #241

Closed
gstanchev opened this issue Sep 21, 2020 · 1 comment
Closed

Switch to SecureRandom for encryption randomness #241

gstanchev opened this issue Sep 21, 2020 · 1 comment
Assignees
@srikanth-lingala
Labels
improvement in-progress

Comments

@gstanchev
Copy link

gstanchev commented Sep 21, 2020 •
edited
Loading

Both StandardEncrypter and AES encrypter user j.u.Random to generate randomness for encryption operations (salt in case of AES and some other random numbers in the header). Please switch to SecureRandom.
@srikanth-lingala srikanth-lingala self-assigned this Sep 22, 2020
srikanth-lingala added a commit that referenced this issue Sep 22, 2020
@srikanth-lingala
#241 Switch to SecureRandom
613279a
srikanth-lingala added a commit that referenced this issue Sep 22, 2020
@srikanth-lingala
#241 Fix NPE
1cd8276
@srikanth-lingala
Copy link
Owner

Issue fixed in v2.6.3 released today.

@dependabot dependabot bot mentioned this issue Mar 18, 2021
Closed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@srikanth-lingala srikanth-lingala

Labels
improvement in-progress
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
2 participants
@gstanchev @srikanth-lingala