You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
Firstly, I'd like to express my gratitude for developing such a useful project!
While perusing the source code, I noticed that this project uses the endpoint https://${this._userPoolDomain}/authorize for Cognito's authorization endpoint. Yet, the official Cognito documentation lists the authorization endpoint as /oauth2/authorize .
Could you shed light on the choice to use /authorize instead of /oauth2/authorize ? While everything seems to be functioning correctly as is, aligning with the official documentation might be beneficial for clarity and potential future-proofing.
If there were specific considerations or historical reasons for this choice during the early implementation phases, I'd be keen to understand.
Should you consider making this adjustment, I'm more than willing to submit a PR and would be glad to have it reviewed at your convenience :)
The text was updated successfully, but these errors were encountered:
How can we help?
Firstly, I'd like to express my gratitude for developing such a useful project!
While perusing the source code, I noticed that this project uses the endpoint
https://${this._userPoolDomain}/authorize
for Cognito's authorization endpoint. Yet, the official Cognito documentation lists the authorization endpoint as/oauth2/authorize
.cognito-at-edge/src/index.ts
Line 547 in e8c7e30
https://docs.aws.amazon.com/cognito/latest/developerguide/authorization-endpoint.html#get-authorize
Could you shed light on the choice to use
/authorize
instead of/oauth2/authorize
? While everything seems to be functioning correctly as is, aligning with the official documentation might be beneficial for clarity and potential future-proofing.If there were specific considerations or historical reasons for this choice during the early implementation phases, I'd be keen to understand.
Should you consider making this adjustment, I'm more than willing to submit a PR and would be glad to have it reviewed at your convenience :)
The text was updated successfully, but these errors were encountered: