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

Trunk 3933 valid username leak random question generator #732

Closed
wants to merge 2 commits into from

Conversation

amberheilman
Copy link

Forgotten Password Form Leaks Valid Usernames simple fix using random question generator.

@ghost
Copy link

ghost commented Mar 7, 2014

Hi @amberheilman :-) In the future, it would probably be best if you can title your pull request with not only the JIRA issue number but with a short description, similar to what was done in commit b5a0787. Thanks!

@amberheilman
Copy link
Author

name is now updated

@ghost
Copy link

ghost commented Mar 7, 2014

Thanks! 👍

@dkayiwa
Copy link
Member

dkayiwa commented Mar 10, 2014

@amberheilman this pull request seems to have some problems. Can you update your code with the latest changes from upstream, then create a new one? Remember to also address the comment i have just put. Also include some unit tests.
This page can be of help for getting the latest changes from upstream: https://wiki.openmrs.org/display/docs/Using+Git under section: 5 Before you pick up your work again, update to the latest code.

@dkayiwa dkayiwa closed this Mar 10, 2014
RandilaP pushed a commit to RandilaP/openmrs-core that referenced this pull request Jul 31, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
2 participants