Skip to content

[RFC-0010] Add provider audience to cache key and decouple packages #928

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

Merged
merged 1 commit into from
May 19, 2025

Conversation

matheuscscp
Copy link
Member

@matheuscscp matheuscscp commented May 17, 2025

Part of: fluxcd/flux2#5022

After #925 the provider audience is no longer unique per provider, hence it needs to be part of the token cache key.

We are also decoupling the auth package from the oci, git and git/gogit packages, which makes it much easier to release changes in auth and also avoids forcing these dependants from including loads of indirect dependencies that are extremely unneeded.

@matheuscscp matheuscscp requested a review from a team as a code owner May 17, 2025 20:48
@matheuscscp matheuscscp force-pushed the auth-audience-cache-key branch from e2be6b8 to 1bda12b Compare May 17, 2025 20:51
@matheuscscp matheuscscp force-pushed the auth-audience-cache-key branch from 1bda12b to 6a42f84 Compare May 17, 2025 22:38
@matheuscscp matheuscscp force-pushed the auth-audience-cache-key branch from 6a42f84 to 61878a4 Compare May 19, 2025 10:53
@matheuscscp matheuscscp changed the title [RFC-0010] Add provider audience to cache key [RFC-0010] Add provider audience to cache key and decouple packages May 19, 2025
@matheuscscp matheuscscp force-pushed the auth-audience-cache-key branch from 61878a4 to 82fd687 Compare May 19, 2025 11:03
@matheuscscp matheuscscp merged commit 69fd2d1 into main May 19, 2025
18 of 20 checks passed
@matheuscscp matheuscscp deleted the auth-audience-cache-key branch May 19, 2025 11:28
@matheuscscp
Copy link
Member Author

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.

2 participants