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/sso signup #1555
Feature/sso signup #1555
Conversation
This pull request is being automatically deployed with Vercel (learn more). 🔍 Inspect: https://vercel.com/cal/calendso/EgvqHoJpAu9rBZcfR4DacXtuf28w |
…his is causing the first record to be retrieved instead of the AND query failing
…(as long as email matches)
# Conflicts: # pages/getting-started.tsx # yarn.lock
merged calcom/website#47 |
Tested Google, works like a charm :) |
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.
Code looks good to me, will test shortly
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.
- The save button in
/settings/security
for SAML xml needs visual update - SAML currently doesn't work for users with existing account, this is not intended.
Everything else works well :)
Thanks @alishaz-polymath. Save button has been fixed now. For existing accounts I block login from another identity provider if the email has not been verified yet (for security reasons). Otherwise it will go through (just re-tested here). |
* updated saml-jackson * if logged in redirect to getting-started page with username in the query param * fixed issue with mixed up Google login, profile.id is undefined and this is causing the first record to be retrieved instead of the AND query failing * updated updated saml-jackson * document PGSSLMODE for Heroku * tweaks to PGSSLMODE doc * for self-hosted instance just allow user to signin with any identity (as long as email matches) * fixed submitting flag * added username to onboarding flow (if requested during signup) * added telemetry for google login, saml login, saml config * check if firstName and lastName are defined * convert mutation to an async op * added e2e test to ensure username query param gets picked up during onboarding * fixed minor typo and added note about configuring Google integration as an Internal app when self-hosting * cleaned up unnecessary ssr in sso signup routes * renamed function * Revert "cleaned up unnecessary ssr in sso signup routes" This reverts commit 3607ffe. * moved client side code to useEffect hook * - format - fixed Save button in SAML config component Co-authored-by: Peer Richelsen <peeroke@gmail.com> Co-authored-by: kodiakhq[bot] <49736102+kodiakhq[bot]@users.noreply.github.com>
To be merged with calendso/website#47
What does this PR do?
Updates saml-jackson and supports Google and SAML signup via a redirect from the Cal.com website. It also allows signing in with multiple identities for the same email (only for self-hosted instances)
Type of change
How should this be tested?
Checklist: