Skip to content

MIDRC-1285 Validate token audience#110

Draft
paulineribeyre wants to merge 13 commits into
masterfrom
validate-aud
Draft

MIDRC-1285 Validate token audience#110
paulineribeyre wants to merge 13 commits into
masterfrom
validate-aud

Conversation

@paulineribeyre

Copy link
Copy Markdown
Contributor

Link to JIRA ticket if there is one: https://ctds-planx.atlassian.net/browse/MIDRC-1285

Reverts changes from #87, except the default audience is now "gen3"

New Features

Breaking Changes

Bug Fixes

Improvements

Dependency updates

Deployment changes

@github-actions

github-actions Bot commented Jun 9, 2026

Copy link
Copy Markdown

The style in this PR agrees with black. ✔️

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

@coveralls

coveralls commented Jun 9, 2026

Copy link
Copy Markdown

Coverage Report for CI Build 27558089821

Coverage increased (+0.2%) to 60.246%

Details

  • Coverage increased (+0.2%) from the base build.
  • Patch coverage: No coverable lines changed in this PR.
  • 20 coverage regressions across 2 files.

Uncovered Changes

No uncovered changes found.

Coverage Regressions

20 previously-covered lines in 2 files lost coverage.

File Lines Losing Coverage Coverage
authutils/token/core.py 10 80.36%
authutils/user.py 10 70.45%

Coverage Stats

Coverage Status
Relevant Lines: 488
Covered Lines: 294
Line Coverage: 60.25%
Coverage Strength: 0.6 hits per line

💛 - Coveralls

Comment thread src/authutils/token/validate.py Outdated
scopes, audience and purpose (all optional).

Args:
audience (Optional[str|set])

@nss10 nss10 Jun 15, 2026

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

audience scope

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