Thanks to visit codestin.com
Credit goes to github.com

Skip to content

Conversation

@ohjonah
Copy link
Contributor

@ohjonah ohjonah commented Dec 16, 2025

  • Introduced AuthenticationMethod type to define various authentication methods.
  • Updated AuthenticationResponse and AuthenticationResponseRaw interfaces to include authenticationMethod.
  • Modified the deserializeAuthenticationResponse function to handle the new authentication_method field.
  • Updated tests to include authenticationMethod in session data setup.

- Introduced `AuthenticationMethod` type to define various authentication methods.
- Updated `AuthenticationResponse` and `AuthenticationResponseRaw` interfaces to include `authenticationMethod`.
- Modified the `deserializeAuthenticationResponse` function to handle the new `authentication_method` field.
- Updated tests to include `authenticationMethod` in session data setup.
Copy link
Collaborator

@cmatheson cmatheson left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

thanks @ohjonah!

import { User, UserRaw } from "./user.interface";
import { Impersonator, ImpersonatorRaw } from "./impersonator.interface";

export type AuthenticationMethod =
Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

the complete list of authentication methods is

SSO
Password
Passkey
AppleOAuth
BitbucketOAuth
CrossAppAuth
DiscordOAuth
ExternalAuth
GitHubOAuth
GitLabOAuth
GoogleOAuth
LinkedInOAuth
MicrosoftOAuth
SalesforceOAuth
SlackOAuth
VercelMarketplaceOAuth
VercelOAuth
XeroOAuth
MagicAuth
Impersonation
MigratedSession

@nholden nholden merged commit 98a34ca into main Dec 23, 2025
4 checks passed
@nholden nholden deleted the add-authentication-method branch December 23, 2025 22:53
@nholden nholden mentioned this pull request Dec 23, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Development

Successfully merging this pull request may close these issues.

4 participants