[Draft]: SSO using nextauth

[bhavanakarwade]

Important

Introduces SSO using NextAuth with redirect handling and updates authentication components to support redirectTo and clientAlias.

• Authentication:
  • Introduces SSO using NextAuth in authOptions with redirect handling.
  • Updates EmailVerificationForm, SignUpUser, UserInfoForm, VerifyEmail, and user-auth-form to handle redirectTo and clientAlias parameters.
  • Modifies SessionCheck to manage session redirects based on redirectTo.
• API Changes:
  • Changes sendVerificationMail in Auth.ts to use clientAlias instead of clientId and clientSecret.
• UI Components:
  • Updates Modal.tsx to improve code formatting.
  • Adjusts PasswordSuggestionBox.tsx for better styling.
• Miscellaneous:
  • Updates authSlice.ts and authOptions.ts for session token management.
  • Refactors OrganizationDashboard, CreateOrganizationModal, and WalletSpinupComponent to handle redirectTo and clientAlias.

^{This description was created by for 784f405. You can customize this summary. It will automatically update as commits are pushed.}

[ellipsis-dev]

Caution

Changes requested ❌

Reviewed everything up to e09bef2 in 2 minutes and 13 seconds. Click for details.

• Reviewed 620 lines of code in 19 files
• Skipped 0 files when reviewing.
• Skipped posting 7 draft comments. View those below.
• Modify your settings and rules to customize what types of comments Ellipsis leaves. And don't forget to react with 👍 or 👎 to teach Ellipsis.

1. nextjs/src/features/auth/components/user-auth-form.tsx:84

• Draft comment:
  Typo in query parameter: 'clienAlias' should be 'clientAlias'.
• Reason this comment was not posted:
  Marked as duplicate.

2. nextjs/src/features/wallet/SetPrivateKeyValue.tsx:78

• Draft comment:
  Ensure consistent private key format. The check for length 64 may not account for a 0x-prefix.
• Reason this comment was not posted:
  Comment was not on a location in the diff, so it can't be submitted as a review comment.

3. nextjs/src/utils/authOptions.ts:74

• Draft comment:
  Possible typo in API route. 'sinIn' might be intended as 'signIn'.
• Reason this comment was not posted:
  Comment was not on a location in the diff, so it can't be submitted as a review comment.

4. nextjs/src/utils/authOptions.ts:62

• Draft comment:
  The credentials provider includes an unused 'isPasskey' field. Consider removing it if not needed.
• Reason this comment was not posted:
  Comment was not on a location in the diff, so it can't be submitted as a review comment.

5. nextjs/src/features/organization/connectionIssuance/components/ConnectionList.tsx:125

• Draft comment:
  Consider using a controlled checkbox instead of defaultChecked for consistency.
• Reason this comment was not posted:
  Comment was not on a location in the diff, so it can't be submitted as a review comment.

6. nextjs/src/features/auth/components/SignUpUser.tsx:20

• Draft comment:
  Typo: The query parameter string uses 'clienAlias' instead of 'clientAlias'.
• Reason this comment was not posted:
  Marked as duplicate.

7. nextjs/src/utils/authOptions.ts:74

• Draft comment:
  Typographical error: It appears that 'sinIn' may be a typo. Should it be 'signIn' for the authentication endpoint?
• Reason this comment was not posted:
  Comment was not on a location in the diff, so it can't be submitted as a review comment.

Workflow ID: wflow_Y8ZJ9bHNd5aqil8o

^{You can customize by changing your verbosity settings, reacting with 👍 or 👎, replying to comments, or adding code review rules.}

[ellipsis-dev]

Important

Looks good to me! 👍

Reviewed 784f405 in 28 seconds. Click for details.

• Reviewed 12 lines of code in 1 files
• Skipped 0 files when reviewing.
• Skipped posting 1 draft comments. View those below.
• Modify your settings and rules to customize what types of comments Ellipsis leaves. And don't forget to react with 👍 or 👎 to teach Ellipsis.

1. nextjs/src/features/profile/components/EditUserProfile.tsx:274

• Draft comment:
  Extra newline added. If this spacing is intentional for readability, it's fine; otherwise, consider removing unnecessary whitespace.
• Reason this comment was not posted:
  Confidence changes required: 0% <= threshold 50% None

Workflow ID: wflow_QLFjaY9qGMuSDz7y

^{You can customize by changing your verbosity settings, reacting with 👍 or 👎, replying to comments, or adding code review rules.}

[sonarqubecloud]

Quality Gate passed

Issues
8 New issues
0 Accepted issues

Measures
0 Security Hotspots
0.0% Coverage on New Code
0.0% Duplication on New Code

See analysis details on SonarQube Cloud