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

[UNDERTOW-2097] DMI_RANDOM_USED_ONLY_ONCE error reported by spotbugs #1325

Merged
merged 1 commit into from Aug 16, 2022

Conversation

jstourac
Copy link
Contributor

@jstourac jstourac commented Jun 1, 2022

Let's try to reuse instance of SecureRandom class when possible. This
change is attempt to fix error identified by SpotBugs plugin and
described in [1].

[1] https://spotbugs.readthedocs.io/en/stable/bugDescriptions.html#dmi-random-object-created-and-used-only-once-dmi-random-used-only-once

https://issues.redhat.com/browse/UNDERTOW-2097

Let's try to reuse instance of `SecureRandom` class when possible. This
change is attempt to fix error identified by SpotBugs plugin and
described in [1].

[1] https://spotbugs.readthedocs.io/en/stable/bugDescriptions.html#dmi-random-object-created-and-used-only-once-dmi-random-used-only-once
@jstourac
Copy link
Contributor Author

just rebased against master branch

@jstourac
Copy link
Contributor Author

Failed tests on JDK17 seem to be the very same as are in other PRs so I suppose they are not tied with my changes here.

@fl4via fl4via added enhancement Enhances existing behaviour or code next release This PR will be merged before next release or has already been merged (for payload double check) labels Aug 5, 2022
@fl4via fl4via merged commit e11eba5 into undertow-io:master Aug 16, 2022
@fl4via fl4via removed the next release This PR will be merged before next release or has already been merged (for payload double check) label Sep 18, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement Enhances existing behaviour or code
Projects
None yet
2 participants