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.

Sign up for GitHub

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

Jump to bottom

Do not cache disabled tokens #7061

Merged
merged 29 commits into from
Aug 6, 2024
Merged

Do not cache disabled tokens #7061

merged 29 commits into from
Aug 6, 2024

Conversation

amontenegro
Copy link
Member

No description provided.

@amontenegro
Deactivated records should get 409 on GET requests
3279481
@amontenegro
Merge branch 'main' of https://github.com/ORCID/ORCID-Source
b3660c2
@amontenegro
Merge branch 'main' of https://github.com/ORCID/ORCID-Source
2cc66ab
@amontenegro
Merge branch 'main' of https://github.com/ORCID/ORCID-Source
f2dc713
@amontenegro
Merge branch 'main' of https://github.com/ORCID/ORCID-Source
b4f8223
@amontenegro
Merge branch 'main' of https://github.com/ORCID/ORCID-Source
b0026c3
@amontenegro
Merge branch 'main' of https://github.com/ORCID/ORCID-Source
53e7616
@amontenegro
Merge branch 'main' of https://github.com/ORCID/ORCID-Source
bf82372
@amontenegro
Merge branch 'main' of https://github.com/ORCID/ORCID-Source
23b6afb
@amontenegro
Merge branch 'main' of https://github.com/ORCID/ORCID-Source
182c67d
@amontenegro
Merge branch 'main' of https://github.com/ORCID/ORCID-Source
c52ef13
@amontenegro
Merge branch 'main' of https://github.com/ORCID/ORCID-Source
d4f779b
@amontenegro
Merge branch 'main' of https://github.com/ORCID/ORCID-Source
27b0033
@amontenegro
Merge branch 'main' of https://github.com/ORCID/ORCID-Source
585b896
@amontenegro
Merge branch 'main' of https://github.com/ORCID/ORCID-Source
9c9dfef
@amontenegro
Merge branch 'main' of https://github.com/ORCID/ORCID-Source
42ac636
@amontenegro
Merge branch 'main' of https://github.com/ORCID/ORCID-Source
3f0d771
@amontenegro
Merge branch 'main' of https://github.com/ORCID/ORCID-Source
e65ec79
@amontenegro
Merge branch 'main' of https://github.com/ORCID/ORCID-Source
e69191a
@amontenegro
Merge branch 'main' of https://github.com/ORCID/ORCID-Source
cff8029
@amontenegro
Merge branch 'main' of https://github.com/ORCID/ORCID-Source
ddf3e67
@amontenegro
Merge branch 'main' of https://github.com/ORCID/ORCID-Source
a091bb3
@amontenegro
Merge branch 'main' of https://github.com/ORCID/ORCID-Source
97024f0
@amontenegro
Merge branch 'main' of https://github.com/ORCID/ORCID-Source
3201e7e
@amontenegro
Merge branch 'main' of https://github.com/ORCID/ORCID-Source
c85de91
@amontenegro
Merge branch 'main' of https://github.com/ORCID/ORCID-Source
3132245
@amontenegro
Merge branch 'main' of https://github.com/ORCID/ORCID-Source
3672cfd
@amontenegro
Do not cache the token if it id disabled, since the token does not kn…
04c185a 
…ow its disabled state
@amontenegro
Remove unused imports
d6027d8
@amontenegro
Copy link
Member Author

[QA] Remove Member OBO tokens from cache when they are revoked

@amontenegro
Copy link
Member Author

@Camelia-Orcid please review this.

Thanks

@amontenegro amontenegro merged commit ee144da into main Aug 6, 2024
24 checks passed
@amontenegro amontenegro deleted the DoNotCacheDisabledTokens branch August 6, 2024 21:16
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
1 participant
@amontenegro