Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

feat(backup): Generate new OrgAuthToken on input #56022

Merged
merged 1 commit into from
Sep 14, 2023

Conversation

azaslavsky
Copy link
Contributor

Instead of making a duplicate of the existing token when we copy an org across regions, which could cause problems for downstream code that relies on this uniqueness, we simply make a new instance of the token value. This will break operations that rely on this token as users move from the old org to the new one, but this an okay price to pay to preserve uniqueness.

Issue: getsentry/team-ospo#193

@github-actions github-actions bot added the Scope: Backend Automatically applied to PRs that change backend components label Sep 11, 2023
@codecov
Copy link

codecov bot commented Sep 11, 2023

Codecov Report

Merging #56022 (dfb047a) into master (2a46053) will decrease coverage by 1.39%.
Report is 5 commits behind head on master.
The diff coverage is 26.92%.

@@            Coverage Diff             @@
##           master   #56022      +/-   ##
==========================================
- Coverage   80.00%   78.61%   -1.39%     
==========================================
  Files        5066     5065       -1     
  Lines      217964   217411     -553     
  Branches    36889    36781     -108     
==========================================
- Hits       174376   170918    -3458     
- Misses      38240    40980    +2740     
- Partials     5348     5513     +165     
Files Changed Coverage
src/sentry/models/projectkey.py ø
src/sentry/testutils/helpers/backups.py 0.00%
src/sentry/models/orgauthtoken.py 28.00%

Base automatically changed from azaslavsky/backup/rename_test_release to master September 11, 2023 19:43
@azaslavsky azaslavsky force-pushed the azaslavsky/backup/org_auth_token branch from dc76ccf to 435bdf3 Compare September 11, 2023 20:54
@azaslavsky azaslavsky force-pushed the azaslavsky/backup/org_auth_token branch from 435bdf3 to 4577593 Compare September 12, 2023 00:15
Instead of making a duplicate of the existing token when we copy an org
across regions, which could cause problems for downstream code that
relies on this uniqueness, we simply make a new instance of the token
value. This will break operations that rely on this token as users move
from the old org to the new one, but this an okay price to pay to
preserve uniqueness.

Issue: getsentry/team-ospo#193
@azaslavsky azaslavsky force-pushed the azaslavsky/backup/org_auth_token branch from 4577593 to dfb047a Compare September 13, 2023 21:18
@azaslavsky azaslavsky marked this pull request as ready for review September 13, 2023 22:30
@azaslavsky azaslavsky requested a review from a team as a code owner September 13, 2023 22:30
Copy link
Member

@markstory markstory left a comment

Choose a reason for hiding this comment

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

Looks good to me.

@azaslavsky azaslavsky merged commit e6abdbc into master Sep 14, 2023
51 of 52 checks passed
@azaslavsky azaslavsky deleted the azaslavsky/backup/org_auth_token branch September 14, 2023 16:05
@github-actions github-actions bot locked and limited conversation to collaborators Sep 30, 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.

2 participants