Update cookie settings. Add cookie env#653
Conversation
|
@edulelis is attempting to deploy a commit to the Inbox Zero OSS Program Team on Vercel. A member of the Team first needs to authorize it. |
|
Caution Review failedThe pull request is closed. WalkthroughThis change updates authentication and environment configuration in the web app. It introduces a new environment variable for cookie domains, dynamically applies this variable to cookie settings, enables account linking for Google and Microsoft providers, adds a new user callback URL for social sign-ins, and disables ID token sign-in for these providers. Changes
Sequence Diagram(s)sequenceDiagram
participant User
participant AddAccountComponent
participant AuthClient
participant AuthConfig
User->>AddAccountComponent: Click "Connect Google/Microsoft"
AddAccountComponent->>AuthClient: signIn.social({callbackURL, newUserCallbackURL, scopes})
AuthClient->>AuthConfig: Use provider config (with disableIdTokenSignIn, accountLinking)
AuthClient->>User: Redirect to provider auth flow
Note right of AuthClient: Cookie domain set from env.COOKIE_DOMAIN
Estimated code review effort🎯 2 (Simple) | ⏱️ ~8 minutes Possibly related PRs
Poem
📜 Recent review detailsConfiguration used: .coderabbit.yaml 📒 Files selected for processing (3)
✨ Finishing Touches
🧪 Generate unit tests
Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out. 🪧 TipsChatThere are 3 ways to chat with CodeRabbit:
SupportNeed help? Create a ticket on our support page for assistance with any issues or questions. CodeRabbit Commands (Invoked using PR comments)
Other keywords and placeholders
Documentation and Community
|
Summary by CodeRabbit
New Features
Improvements