Navigation Menu

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

Feature :: Email Verifcation #5372

Merged
merged 21 commits into from Nov 23, 2022
Merged

Feature :: Email Verifcation #5372

merged 21 commits into from Nov 23, 2022

Conversation

rahul-rocket
Copy link
Collaborator

@rahul-rocket rahul-rocket commented Nov 21, 2022

PR

Please note: we will close your PR without comment if you do not check the boxes above and provide ALL requested information.


/**
* Email verification options
*/
JWT_VERIFICATION_TOKEN_SECRET: process.env.JWT_VERIFICATION_TOKEN_SECRET || 'verificationSecretKey',

Check failure

Code scanning / CodeQL

Hard-coded credentials

The hard-coded value "verificationSecretKey" is used as [key](1).
@lgtm-com
Copy link

lgtm-com bot commented Nov 21, 2022

This pull request introduces 1 alert when merging 64397d5 into e1807ff - view on LGTM.com

new alerts:

  • 1 for Hard-coded credentials

Heads-up: LGTM.com's PR analysis will be disabled on the 5th of December, and LGTM.com will be shut down ⏻ completely on the 16th of December 2022. Please enable GitHub code scanning, which uses the same CodeQL engine ⚙️ that powers LGTM.com. For more information, please check out our post on the GitHub blog.

@lgtm-com
Copy link

lgtm-com bot commented Nov 21, 2022

This pull request introduces 1 alert when merging 30c3847 into e1807ff - view on LGTM.com

new alerts:

  • 1 for Hard-coded credentials

Heads-up: LGTM.com's PR analysis will be disabled on the 5th of December, and LGTM.com will be shut down ⏻ completely on the 16th of December 2022. Please enable GitHub code scanning, which uses the same CodeQL engine ⚙️ that powers LGTM.com. For more information, please check out our post on the GitHub blog.

@rahul-rocket rahul-rocket changed the title feat: email verified column for user table Feature :: Email Verifcation Nov 21, 2022
@lgtm-com
Copy link

lgtm-com bot commented Nov 22, 2022

This pull request introduces 1 alert when merging 69963d3 into e1807ff - view on LGTM.com

new alerts:

  • 1 for Hard-coded credentials

Heads-up: LGTM.com's PR analysis will be disabled on the 5th of December, and LGTM.com will be shut down ⏻ completely on the 16th of December 2022. Please enable GitHub code scanning, which uses the same CodeQL engine ⚙️ that powers LGTM.com. For more information, please check out our post on the GitHub blog.

@lgtm-com
Copy link

lgtm-com bot commented Nov 22, 2022

This pull request introduces 1 alert when merging fad0afa into 658e30a - view on LGTM.com

new alerts:

  • 1 for Hard-coded credentials

Heads-up: LGTM.com's PR analysis will be disabled on the 5th of December, and LGTM.com will be shut down ⏻ completely on the 16th of December 2022. Please enable GitHub code scanning, which uses the same CodeQL engine ⚙️ that powers LGTM.com. For more information, please check out our post on the GitHub blog.

@rahul-rocket rahul-rocket marked this pull request as ready for review November 23, 2022 09:21
@lgtm-com
Copy link

lgtm-com bot commented Nov 23, 2022

This pull request introduces 1 alert when merging a4c6529 into 02b4da5 - view on LGTM.com

new alerts:

  • 1 for Hard-coded credentials

Heads-up: LGTM.com's PR analysis will be disabled on the 5th of December, and LGTM.com will be shut down ⏻ completely on the 16th of December 2022. Please enable GitHub code scanning, which uses the same CodeQL engine ⚙️ that powers LGTM.com. For more information, please check out our post on the GitHub blog.

@lgtm-com
Copy link

lgtm-com bot commented Nov 23, 2022

This pull request introduces 1 alert when merging 77e64e5 into 02b4da5 - view on LGTM.com

new alerts:

  • 1 for Hard-coded credentials

Heads-up: LGTM.com's PR analysis will be disabled on the 5th of December, and LGTM.com will be shut down ⏻ completely on the 16th of December 2022. Please enable GitHub code scanning, which uses the same CodeQL engine ⚙️ that powers LGTM.com. For more information, please check out our post on the GitHub blog.

@lgtm-com
Copy link

lgtm-com bot commented Nov 23, 2022

This pull request introduces 1 alert when merging a2f0414 into 02b4da5 - view on LGTM.com

new alerts:

  • 1 for Hard-coded credentials

Heads-up: LGTM.com's PR analysis will be disabled on the 5th of December, and LGTM.com will be shut down ⏻ completely on the 16th of December 2022. Please enable GitHub code scanning, which uses the same CodeQL engine ⚙️ that powers LGTM.com. For more information, please check out our post on the GitHub blog.

@evereq evereq merged commit 45218bd into develop Nov 23, 2022
@evereq evereq deleted the fix/email-confirmation branch November 23, 2022 11:19
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants