Skip to content

Commit fb92ee2

Browse files
boehlkeclaude
andcommitted
fix: pass internalIssuerURL to NewOIDCValidator in tests
NewOIDCValidator now takes 3 args (issuerURL, internalIssuerURL, clientID). Update all test call sites to pass issuerURL as both external and internal URL. Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
1 parent 328740e commit fb92ee2

1 file changed

Lines changed: 3 additions & 3 deletions

File tree

auth/oidc_test.go

Lines changed: 3 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -43,7 +43,7 @@ func TestOIDCValidator(t *testing.T) {
4343
issuerURL := jwksServer.URL
4444
clientID := "test-client"
4545

46-
validator := auth.NewOIDCValidator(issuerURL, clientID)
46+
validator := auth.NewOIDCValidator(issuerURL, issuerURL, clientID)
4747

4848
t.Run("ValidToken", func(t *testing.T) {
4949
token := createTestOIDCToken(t, privateKey, issuerURL, clientID, "keycloak-user-123")
@@ -142,7 +142,7 @@ func TestOIDCValidatorKeyCache(t *testing.T) {
142142
}))
143143
defer jwksServer.Close()
144144

145-
validator := auth.NewOIDCValidator(jwksServer.URL, "test-client")
145+
validator := auth.NewOIDCValidator(jwksServer.URL, jwksServer.URL, "test-client")
146146

147147
// Validate multiple tokens - should only fetch JWKS once
148148
for i := 0; i < 5; i++ {
@@ -166,7 +166,7 @@ func TestOIDCTestServer(t *testing.T) {
166166
}
167167
defer ots.Close()
168168

169-
validator := auth.NewOIDCValidator(ots.IssuerURL, ots.ClientID)
169+
validator := auth.NewOIDCValidator(ots.IssuerURL, ots.IssuerURL, ots.ClientID)
170170

171171
t.Run("CreateToken", func(t *testing.T) {
172172
token, err := ots.CreateToken("user-123")

0 commit comments

Comments
 (0)