Skip to content

fix: NewRemoteKeySet is not using DefaultHTTPClient #3197

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
Merged
tuunit merged 2 commits into from
Nov 11, 2025
Merged

fix: NewRemoteKeySet is not using DefaultHTTPClient #3197

tuunit merged 2 commits into from
Nov 11, 2025
+5 −0

Conversation

@rsrdesarrollo
Copy link
Contributor

@rsrdesarrollo rsrdesarrollo commented Sep 11, 2025 •
edited by tuunit
Loading

Description

fix #3196 as described in issue

Motivation and Context

How Has This Been Tested?

Checklist:

  • My change requires a change to the documentation or CHANGELOG.
  • I have updated the documentation/CHANGELOG accordingly.
  • I have created a feature (non-master) branch for my PR.
  • I have written tests for my code changes.

@rsrdesarrollo rsrdesarrollo requested a review from a team as a code owner September 11, 2025 12:57
@github-actions github-actions bot added the go label Sep 11, 2025
tuunit
tuunit reviewed Sep 25, 2025
View reviewed changes
@tuunit tuunit changed the title fix #3196 fix: NewRemoteKeySet is not using DefaultHTTPClient Oct 28, 2025
@github-actions github-actions bot added the docs label Nov 11, 2025
tuunit
tuunit previously approved these changes Nov 11, 2025
View reviewed changes
rsrdesarrollo and others added 2 commits November 11, 2025 21:04
@rsrdesarrollo @tuunit
fix: NewRemoteKeySet is not using DefaultHTTPClient
84d82b0 
Signed-off-by: Jan Larwig <jan@larwig.com>
@tuunit
doc: add changelog entry
35ed208 
Signed-off-by: Jan Larwig <jan@larwig.com>
@tuunit tuunit merged commit 7cf69b2 into oauth2-proxy:master Nov 11, 2025
4 checks passed
dvmartinweigl pushed a commit to datavisyn/oauth2-proxy that referenced this pull request Nov 20, 2025
@rsrdesarrollo @tuunit @dvmartinweigl
fix: NewRemoteKeySet is not using DefaultHTTPClient (oauth2-proxy#3197)
20aad14 
* fix: NewRemoteKeySet is not using DefaultHTTPClient

Signed-off-by: Jan Larwig <jan@larwig.com>

* doc: add changelog entry

Signed-off-by: Jan Larwig <jan@larwig.com>

---------

Signed-off-by: Jan Larwig <jan@larwig.com>
Co-authored-by: Jan Larwig <jan@larwig.com>
Signed-off-by: dvmartinweigl <martin.weigl@datavisyn.io>
@BrewTestBot BrewTestBot mentioned this pull request Jan 17, 2026
Merged
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@tuunit tuunit tuunit left review comments

Assignees

No one assigned

Labels

docs go

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

[Bug]: NewRemoteKeySet is not using DefaultHTTPClient

2 participants

@rsrdesarrollo @tuunit