fix: missing server_metadata

[Bento007]

Reason for Change

rolling back to previously working version of authlib.
Previously it was upgraded here a12556f997d9caa1cfbd52453446c5c4af460df3m which caused 502 on the frontend.

Changes

• add server_metadata to oath config
• roll back to previous authlib version.

Testing steps

• verify you can login on frontend
• all automated tests pass.

[github-actions]

Deployment Summary

• backend_url: https://pr-7346-backend.rdev.single-cell.czi.technology
• backend_de_url: https://pr-7346-backend-de.rdev.single-cell.czi.technology
• backend_wmg_url: https://pr-7346-backend-wmg.rdev.single-cell.czi.technology
• frontend_url: https://pr-7346-frontend.rdev.single-cell.czi.technology
• delete_db_task_definition_arn: arn:aws:ecs:us-west-2:***:task-definition/dp-rdev-pr-7346-deletion:1
• migrate_db_task_definition_arn: arn:aws:ecs:us-west-2:***:task-definition/dp-rdev-pr-7346-migration:1

[codecov]

Codecov Report

Attention: Patch coverage is 50.00000% with 2 lines in your changes missing coverage. Please review.

Project coverage is 92.92%. Comparing base (fda46d2) to head (bbe151a).

Files with missing lines	Patch %	Lines
backend/common/census_cube/utils.py	33.33%	2 Missing ⚠️

Additional details and impacted files

@@           Coverage Diff           @@
##             main    #7346   +/-   ##
=======================================
  Coverage   92.91%   92.92%           
=======================================
  Files         194      194           
  Lines       16827    16828    +1     
=======================================
+ Hits        15635    15637    +2     
+ Misses       1192     1191    -1     

Flag	Coverage Δ
unittests	92.92% <50.00%> (+<0.01%)	⬆️

Flags with carried forward coverage won't be shown. Click here to find out more.

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

[nayib-jose-gloria]

If we're rolling back the authlib change, why do we still need to add in server metadata when its working without it?

[Bento007]

it will be needed in the future anyways according to auth0 documentation.

[nayib-jose-gloria]

it will be needed in the future anyways according to auth0 documentation.

I'm still getting a 502 gateway when i try to log in on rdev

[Bento007]

Still looking for where that 502 is coming from.

• rdev does not use cloudfront
• I did not find any 502 in the logs for the contianer dataportal-rdev-happy-proxy
• I can see the callback in
  pr-7346-backend and it's returning 302.