Work on the Authentication

[shazm12]

I need someone to work on the authentication logic of my web app. Currently I am just validating the data given using simple checks and then encrypting my passwords before storing to the DB. I am thinking to use JWT Tokens or passport.js for better and secure authentication flow. What are your takes on this? Can you implement this system?
Will check the following points

• Secure and working Authentication flow.
• Libraries and components used
• How well you maintain code

Rest is up to your creativity!

[meetagrawal09]

Hey @shazm12, I would like to work on this. Can you assign this to me
I am planning to do it using JWT authentication
The idea here will be that all the APIs will be secured and will be accessible using the token
My implementation includes creating a token generator and further a middleware to validate the token

[shazm12]

sounds amazing @meetagrawal09 I am assigning this issue to you.

[meetagrawal09]

@shazm12,
You have not included a .gitignore file, anyone who is to clone the repository also gets in node_modules alongside
Also there are some small typos in the code which cause the build to fail initially

[meetagrawal09]

@shazm12, can you please explain me the need of a package.json outside client and server folders. Because I see that your package.json in the root directory has some UI libraries included like material-ui which could have been put up inside of client directory

[shazm12]

that's not

@shazm12, You have not included a .gitignore file, anyone who is to clone the repository also gets in node_modules alongside Also there are some small typos in the code which cause the build to fail initially

ohh i thought i had included the .gitignore file so sorry for this, yes the proposed changes makes sense please do and make a PR i ll label them too as contribution for the hacktoberfest. Thanks!

[shazm12]

@shazm12, can you please explain me the need of a package.json outside client and server folders. Because I see that your package.json in the root directory has some UI libraries included like material-ui which could have been put up inside of client directory

There's no need for it but you might get error in client side the other team mate with me had screwed a little bit with project but mostly without the mentioned package.json file it will work, please check n tell me okay thanks!

[shazm12]

the material-ui was removed from the final version so it should still work

[meetagrawal09]

@shazm12 , raised a PR with all the corrections and some additional security measures
#5

[shazm12]

cool will check it