Skip to content
This repository was archived by the owner on Mar 13, 2025. It is now read-only.

Update to use central lib for auth #347

Closed
jmgasper opened this issue Oct 6, 2020 · 2 comments
Closed

Update to use central lib for auth #347

jmgasper opened this issue Oct 6, 2020 · 2 comments

Comments

@jmgasper
Copy link
Collaborator

jmgasper commented Oct 6, 2020

From #344

Can we use our central lib for auth (instead of https://github.com/topcoder-platform/topcoder-x-ui/tree/develop/src/front/src/app/auth). Example- https://github.com/appirio-tech/connect-app/blob/test/auth0lib/package.json#L153 appirio-tech/connect-app#4082

For the challenge

The expectation is that the new RS256 auth flow will work as expected, while still being able to handle HS256 tokens in the cookies for authorization (if the user logged in on an old app that hasn't been updated yet).

For testing:

  1. Login to TC-X and see the new login form. App should work as normal
  2. After logging in to TC-X, try to navigate to another app and ensure you stay logged in
  3. Logout and into an updated app, like topcoder-dev.com or vanilla.topcoder-dev.com and get an RS256 token
  4. Ensure you can come back to Topcoder X and the cookie is processed and you are logged in
  5. Logout and login and get an old token (https://accounts.topcoder-dev.com/member?retUrl=https:%2F%2Fx.topcoder-dev.com%2F
  6. Ensure you can still navigate Topcoder X and the old token is processed and you are logged in
@jmgasper
Copy link
Collaborator Author

jmgasper commented Oct 6, 2020

Contest https://www.topcoder.com/challenges/30144600 has been created for this ticket.

This is an automated message for ghostar via Topcoder X

@jmgasper
Copy link
Collaborator Author

jmgasper commented Oct 6, 2020

Contest https://www.topcoder.com/challenges/30144600 has been updated - it has been assigned to afrisalyp.

This is an automated message for ghostar via Topcoder X

@jmgasper jmgasper changed the title [$50] Update to use central lib for auth Update to use central lib for auth Oct 20, 2020
@jmgasper jmgasper closed this as completed Jan 3, 2021
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

No branches or pull requests

2 participants