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

Enforce Necessary Parameters for Third-party Sign In Integrations #720

Open
RobSchilderr opened this issue Oct 16, 2023 · 0 comments
Open

Comments

@RobSchilderr
Copy link

Issue:

In the process of upgrading from V14 to V15 of SuperTokens Node and integrating Apple Sign In, it was found that the absence of the redirectURIOnProviderDashboard parameter in the
ThirdPartyPasswordless.getThirdPartyAuthorisationURLWithQueryParamsAndSetState method didn’t trigger a TypeScript error.

This oversight led to a malfunction in the Apple Sign In integration, making it challenging to pinpoint the root cause of the issue due to the lack of immediate feedback from TypeScript.

The redirectURIOnProviderDashboard parameter proved to be crucial for the successful operation of the Apple Sign In integration. When this parameter was included, the integration functioned as expected.

Currently, TypeScript settings do not enforce or flag the missing redirectURIOnProviderDashboard parameter, which could potentially lead to debugging challenges for developers, especially when integrating third-party sign-ins.

Resolution Suggestion:

It is recommended to enforce the inclusion of the redirectURIOnProviderDashboard parameter within the getThirdPartyAuthorisationURLWithQueryParamsAndSetState method, especially for Apple Sign In integration, to prevent similar issues in the future. Enforcing this parameter through TypeScript can provide immediate feedback to developers, aiding in ensuring the correct configuration for a successful integration.

This enforcement should be applied at least for Apple Sign In integration. It's unclear if similar enforcement is needed for other third-party providers.

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

No branches or pull requests

1 participant