Skip to content

Conversation

@mydea
Copy link
Member

@mydea mydea commented Jun 21, 2023

This PR adjusts the token format of the new org auth tokens from using JWT to a custom format based on base64 encoding.
This should result in nicer endings of tokens, and also makes the tokens considerably shorter.

Related to getsentry/rfcs#105

@mydea mydea self-assigned this Jun 21, 2023
@github-actions github-actions bot added the Scope: Backend Automatically applied to PRs that change backend components label Jun 21, 2023
@codecov
Copy link

codecov bot commented Jun 21, 2023

Codecov Report

Merging #51341 (bef17fc) into master (88755b9) will not change coverage.
The diff coverage is 100.00%.

Additional details and impacted files
@@           Coverage Diff           @@
##           master   #51341   +/-   ##
=======================================
  Coverage    81.29    81.29           
=======================================
  Files        4901     4901           
  Lines      205524   205528    +4     
  Branches    11081    11081           
=======================================
  Hits       167076   167076           
- Misses      38203    38207    +4     
  Partials      245      245           
Impacted Files Coverage Δ
src/sentry/api/endpoints/org_auth_tokens.py 92.85% <100.00%> (ø)
src/sentry/utils/security/orgauthtoken_token.py 100.00% <100.00%> (ø)

... and 3 files with indirect coverage changes

@mydea mydea force-pushed the fn/org-token-base64 branch from 7ecd574 to bef17fc Compare June 26, 2023 09:20
@mydea mydea merged commit 90a92cf into master Jun 26, 2023
@mydea mydea deleted the fn/org-token-base64 branch June 26, 2023 09:52
@github-actions github-actions bot locked and limited conversation to collaborators Jul 11, 2023
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

Scope: Backend Automatically applied to PRs that change backend components

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants