Skip to content

Comments

fix(mozilla): moving to mozilla auth proxy#840

Merged
jpetto merged 2 commits intomainfrom
update-mozilla-auth
Jul 2, 2024
Merged

fix(mozilla): moving to mozilla auth proxy#840
jpetto merged 2 commits intomainfrom
update-mozilla-auth

Conversation

@bassrock
Copy link
Contributor

@bassrock bassrock commented Jun 23, 2022

Goal

Move to Mozilla Auth Proxy instead of our older cognito proxy. this should ameliorate the problem we've faced for years when folks are in too many mozilla groups and access breaks (due to the group size breaking the auth payload size restrictions).

Note

the client id value in your local .env file will need to be updated! ask in slack (or check AWS cognito) for the new value!

@jpetto
Copy link
Collaborator

jpetto commented Jun 13, 2024

did a push to dev and am getting the following error (in a private window with a fresh SSO login):

message: "Invalid token specified"
​
stack: "@https://curation-admin-tools.getpocket.dev/static/js/2.3a6bd141.chunk.js:2:956002\ni@https://curation-admin-tools.getpocket.dev/:1:1323\n@https://curation-admin-tools.getpocket.dev/static/js/2.3a6bd141.chunk.js:2:371495\n@https://curation-admin-tools.getpocket.dev/static/js/2.3a6bd141.chunk.js:2:377607\ni@https://curation-admin-tools.getpocket.dev/:1:1323\n833@https://curation-admin-tools.getpocket.dev/static/js/main.2669d016.chunk.js:1:12761\ni@https://curation-admin-tools.getpocket.dev/:1:1323\nr@https://curation-admin-tools.getpocket.dev/:1:1192\nt@https://curation-admin-tools.getpocket.dev/:1:1055\n@https://curation-admin-tools.getpocket.dev/static/js/main.2669d016.chunk.js:1:89\n"

will try to dig in a bit after this work week.

@jpetto jpetto force-pushed the update-mozilla-auth branch from 4c6d1d5 to 44c7d4e Compare June 27, 2024 17:10
@jpetto
Copy link
Collaborator

jpetto commented Jun 27, 2024

just pushed this to dev again and it is working!

@jpetto jpetto marked this pull request as ready for review June 27, 2024 17:16
@jpetto jpetto requested a review from a team as a code owner June 27, 2024 17:16
@jpetto jpetto merged commit fa15939 into main Jul 2, 2024
@jpetto jpetto deleted the update-mozilla-auth branch July 2, 2024 14:53
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

Successfully merging this pull request may close these issues.

3 participants