Skip to content

Fix javadoc in crypto/encrypt/Encryptors.java #9537

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

Merged
merged 1 commit into from
Apr 5, 2021

Conversation

prashanttholia
Copy link
Contributor

Fix 'See Also' section link issue in Encryptors.standard method.

Fix link issue in 'See Also' section of Encryptors.standard method
@spring-projects-issues spring-projects-issues added the status: waiting-for-triage An issue we've not yet triaged label Apr 2, 2021
@prashanttholia
Copy link
Contributor Author

Hi, I made a mistake here. I visited https://docs.spring.io/spring-security/site/docs/4.2.20.RELEASE/apidocs/org/springframework/security/crypto/encrypt/Encryptors.html and corresponding sections in other "RELEASE" versions. The "See Also" section for Encryptors.standard method did not carry a hyperlink. I presumed it to be occurring due to partially qualified name used with the @see tag. I do not know the actual reason and have not been able to find it.
Kindly let me know if I shall close the PR.
I apologize for inconvenience.

@jzheaux jzheaux merged commit 67fdac1 into spring-projects:master Apr 5, 2021
@jzheaux
Copy link
Contributor

jzheaux commented Apr 5, 2021

Thanks for the PR, @prashanttholia! This is now merged into master.

@jzheaux jzheaux self-assigned this Apr 5, 2021
@jzheaux jzheaux added in: crypto An issue in spring-security-crypto type: bug A general bug and removed status: waiting-for-triage An issue we've not yet triaged labels Apr 5, 2021
@jzheaux jzheaux added this to the 5.5.0-RC1 milestone Apr 5, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
in: crypto An issue in spring-security-crypto type: bug A general bug
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants