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

Google auth added #93

Merged
merged 5 commits into from
Oct 3, 2023
Merged

Conversation

Naresh-chandanbatve
Copy link
Collaborator

@Naresh-chandanbatve Naresh-chandanbatve commented Oct 3, 2023

if user uses /auth/google route

first it will check the db for user if it exist it will return the userdata
if it does not exist then it will create new user with the data and then return the
new user data in response as well as add the data in the session with token

also removed redundant imports and models

@vercel
Copy link

vercel bot commented Oct 3, 2023

Someone is attempting to deploy a commit to a Personal Account owned by @UBA-GCOEN on Vercel.

@UBA-GCOEN first needs to authorize it.

@vercel
Copy link

vercel bot commented Oct 3, 2023

The latest updates on your projects. Learn more about Vercel for Git ↗︎

Name Status Preview Comments Updated (UTC)
arrow-server ✅ Ready (Inspect) Visit Preview 💬 Add feedback Oct 3, 2023 6:05pm

Comment on lines +11 to +15
session({
secret: process.env.SESSION_SECRET,
resave: false,
saveUninitialized: true
})

Check failure

Code scanning / CodeQL

Missing CSRF middleware High

This cookie middleware is serving a
request handler
without CSRF protection.
This cookie middleware is serving a
request handler
without CSRF protection.
This cookie middleware is serving a request handler without CSRF protection.
Comment on lines +11 to +15
session({
secret: process.env.SESSION_SECRET,
resave: false,
saveUninitialized: true
})

Check warning

Code scanning / CodeQL

Clear text transmission of sensitive cookie Medium

Sensitive cookie sent without enforcing SSL encryption.
@Siddhant-Patil0203
Copy link
Collaborator

@Naresh-chandanbatve
there is a deployment issue
image
image

Please fix this

@Naresh-chandanbatve Naresh-chandanbatve linked an issue Oct 3, 2023 that may be closed by this pull request
3 tasks
@UBA-GCOEN UBA-GCOEN merged commit 5f6c5a2 into UBA-GCOEN:main Oct 3, 2023
3 of 6 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Feat: add google oauth registrion
3 participants