-
Notifications
You must be signed in to change notification settings - Fork 265
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
Added backend authendication using Mongo node and express #341
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Great job, @Abhishek-TG18! 🎉 Thank you for submitting your pull request. Your contribution is valuable and we appreciate your efforts to improve our project.We will promptly review your changes and offer feedback. Keep up the excellent work! Kindly remember to check our contributing guidelines
✅ Deploy Preview for rococo-mooncake-0d89f7 ready!
To edit notification comments on pull requests, go to your Netlify site configuration. |
|
||
const signUp = async (req, res, next) => { | ||
const { name, email, password, confirmPassword } = req.body; | ||
console.log(name , email,password,confirmPassword) |
Check failure
Code scanning / CodeQL
Clear-text logging of sensitive information High
an access to password
|
||
const signUp = async (req, res, next) => { | ||
const { name, email, password, confirmPassword } = req.body; | ||
console.log(name , email,password,confirmPassword) |
Check failure
Code scanning / CodeQL
Clear-text logging of sensitive information High
an access to confirmPassword
|
||
const signIn = async (req, res, next) => { | ||
const { email, password } = req.body; | ||
console.log(email,password) |
Check failure
Code scanning / CodeQL
Use of externally-controlled format string High
user-provided value
|
||
const signIn = async (req, res, next) => { | ||
const { email, password } = req.body; | ||
console.log(email,password) |
Check failure
Code scanning / CodeQL
Clear-text logging of sensitive information High
an access to password
.findOne({ | ||
}) |
Check failure
Code scanning / CodeQL
Database query built from user-controlled sources High
user-provided value
} = require("../controller/authController.js"); | ||
|
||
authRouter.post("/signup", signUp); | ||
authRouter.post("/signin", signIn); |
Check failure
Code scanning / CodeQL
Missing rate limiting High
a database access
authRouter.post("/signin", signIn); | ||
|
||
|
||
authRouter.get("/user", jwtAuth, getUser); |
Check failure
Code scanning / CodeQL
Missing rate limiting High
authorization
authRouter.post("/signin", signIn); | ||
|
||
|
||
authRouter.get("/user", jwtAuth, getUser); |
Check failure
Code scanning / CodeQL
Missing rate limiting High
a database access
|
||
|
||
authRouter.get("/user", jwtAuth, getUser); | ||
authRouter.get("/logout", jwtAuth, logout); |
Check failure
Code scanning / CodeQL
Missing rate limiting High
authorization
@Abhishek-TG18 i need to implement those methods mentioned in video by u right? |
www.linkedin.com/in/lakshmi-pavananjali-pamidi-766760296 please ping me in linkedin |
@Abhishek-TG18 code scanning results shows failed y? |
🎉 Your pull request has been successfully merged! 🎉 Thank you for your contribution to our project. Your efforts are greatly appreciated. Keep up the fantastic work! 🚀 |
@Abhishek-TG18 our repo is of files HTML CSS JS only in the above pic , main is index.js |
DON'T worry. That I have console some information related to sign in |
Please once see LinkedIn |
Pull Request Format
PR Title
Issue #314 : Implement Authentication using MongoDB, Node.js, and Express.js [backend] solved
Type of PR
Description
This pull request includes the implementation of the backend functionality for user authentication and management. The following features have been added:
All routes and logic have been implemented and tested with Postman. Please review and merge.
Checklist
Information to Start the server
Feel free to ask if you have any doubt
I have tested the code using Postman API Testing here are the demo videos
Post.data.-.My.Workspace.-.Google.Chrome.2024-10-02.16-33-19.mp4
Post.data.-.My.Workspace.-.Google.Chrome.2024-10-02.16-32-32.mp4
Get.data.-.My.Workspace.-.Google.Chrome.2024-10-02.16-48-36.mp4
Get.data.-.My.Workspace.-.Google.Chrome.2024-10-02.16-48-10.mp4
Thank you for reviewing my pull request!