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

[ELY-2601] declared newMap on a separate line in PasswordKeyStoreSpi #1980

Merged
merged 1 commit into from
Oct 2, 2023

Conversation

nasonawa
Copy link
Contributor

@nasonawa nasonawa commented Sep 26, 2023

ELY-2601 newMap on a separate line in PasswordKeyStoreSpi

@PrarthonaPaul
Copy link
Contributor

Hi @nasonawa
Thank you for the PR!
Would you be able to add the ELY issue number to the commit message? You can use git commit --ammend to edit your commit message.
Additionally, please add the link to the ELY issue in the description.

@nasonawa
Copy link
Contributor Author

Hi @PrarthonaPaul
I have made the suggested Changes,
let me know if it's correct or not

Copy link
Contributor

@PrarthonaPaul PrarthonaPaul left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Looks good to me!
Thanks @nasonawa

@@ -91,7 +91,9 @@ public void engineSetEntry(final String alias, final KeyStore.Entry entry, final
if (protParam != null) {
throw log.keyCannotBeProtected(alias);
}
HashMap<String, PasswordEntry> map, newMap;
HashMap<String, PasswordEntry> map;
HashMap<String,PasswordEntry> newMap;
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@nasonawa Justa t total minor, can you please fix the formatting by adding a space before the PasswordEntry of newMap initialization? Thank you!

@nasonawa
Copy link
Contributor Author

Hi @Skyllarr
made suggested the changes.
let me know if it's correct or not

@Skyllarr
Copy link
Contributor

@nasonawa Looks great, thank you!

@Skyllarr Skyllarr merged commit f173144 into wildfly-security:2.x Oct 2, 2023
3 checks passed
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.

3 participants