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

Hashing password #50

Merged

adityabisht02 merged 1 commit into adityabisht02:main from Hares27:patch-1

Oct 23, 2022

Contributor

Hares27 commented Oct 23, 2022

PR Title

The purpose of this Pull Request is to fix #

Postman Screenshots

Checklist

I have tested in local Environment.
I have made the fix as per issue converstaion


          Update users.js

ef034aa

Contributor Author

Hares27 commented Oct 23, 2022

I assume the password must be stored in body of request.

Owner

adityabisht02 commented Oct 23, 2022

What about salt?

Contributor Author

Hares27 commented Oct 23, 2022

There are two ways of giving salt value. I hardcoded that one(one way of giving salt).Everyone use this way only.

adityabisht02 merged commit 24d9b91 into adityabisht02:main

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment