Skip to content

Conversation

@inteon
Copy link
Contributor

@inteon inteon commented Jan 21, 2026

Adds datagatherer that fetches the /.well-known/openid-configuration and /openid/v1/jwks endpoints.

Adding this gathered data to the disco upload is for a follow-up PR.

@inteon inteon force-pushed the oidc_datagatherer branch 3 times, most recently from 539b172 to b4e4169 Compare January 22, 2026 16:41
@inteon inteon added the test-e2e To signal e2e test job to be run label Jan 22, 2026
@inteon inteon closed this Jan 22, 2026
@inteon inteon reopened this Jan 22, 2026
Signed-off-by: Tim Ramlot <42113979+inteon@users.noreply.github.com>
@inteon inteon force-pushed the oidc_datagatherer branch from b4e4169 to 5fa2201 Compare January 23, 2026 08:31
Copy link
Contributor

@SgtCoDFish SgtCoDFish left a comment

Choose a reason for hiding this comment

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

Few minor bits!

Comment on lines +51 to +53
func (g *DataGathererOIDC) Run(ctx context.Context) error {
return nil
}
Copy link
Contributor

Choose a reason for hiding this comment

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

suggestion: probably worth having a comment here explaining why it does nothing!

Copy link
Contributor

Choose a reason for hiding this comment

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

Am I missing the change here? It still seems like there's no comment?

Copy link
Contributor Author

Choose a reason for hiding this comment

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

I added var _ datagatherer.DataGatherer = &DataGathererOIDC{} to make it clear that this struct is implementing an interface.

@inteon inteon requested a review from SgtCoDFish January 23, 2026 10:37
Signed-off-by: Tim Ramlot <42113979+inteon@users.noreply.github.com>
@inteon inteon force-pushed the oidc_datagatherer branch from 75bf546 to 2a994be Compare January 23, 2026 11:10
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

test-e2e To signal e2e test job to be run

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants