Skip to content

Conversation

@paulineribeyre
Copy link
Contributor

@paulineribeyre paulineribeyre commented Jan 8, 2025

Goes with uc-cdis/sheepdog#425

Improvements

  • Support client_credentials tokens: do not assume tokens have a 'sub' field

@github-actions
Copy link

github-actions bot commented Jan 8, 2025

Pull Request Test Coverage Report for Build 12677111463

Details

  • 0 of 0 changed or added relevant lines in 0 files are covered.
  • No unchanged relevant lines lost coverage.
  • Overall coverage increased (+0.04%) to 60.246%

Totals Coverage Status
Change from base Build 9522521071: 0.04%
Covered Lines: 294
Relevant Lines: 488

💛 - Coveralls

@github-actions
Copy link

github-actions bot commented Jan 8, 2025

The style in this PR agrees with black. ✔️

This formatting comment was generated automatically by a script in uc-cdis/wool.

@paulineribeyre paulineribeyre merged commit ee824a4 into master Jan 9, 2025
6 checks passed
@paulineribeyre paulineribeyre deleted the support-client-creds branch January 9, 2025 16:18
paulineribeyre added a commit that referenced this pull request Jan 9, 2025
* feat(deps): upgrade authlib to latest version (major version update)

* travis fix & ignore vscode stuff

* remove period

* travis fix 2

* fix typo

* update ubuntu travis image to resolve openssl issue

* update dependency versions

* update revision number and add comment regarding flask

* fix revision number

* (PPS-294) bump cryptography from 41.0.4 to 42.0.2

* Add debug log + use GH actions (#78)

* Support client_credentials tokens (#79)

* Update version

---------

Co-authored-by: Alex VanTol <avantol@uchicago.edu>
Co-authored-by: tianj7 <jtian1@uchicago.edu>
Co-authored-by: Jian <52763034+tianj7@users.noreply.github.com>
Co-authored-by: George Thomas <george42@uchicago.edu>
Co-authored-by: George Thomas <98996322+george42-ctds@users.noreply.github.com>
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