fixes #2341 Dynamic loading jwk with kid is not working if multiple serviceIds are provided (#2342)

stevehu · web-flow · commit 6a0b587ddbbb · 2024-09-18T21:32:15.000-04:00
diff --git a/security-config/src/main/java/com/networknt/security/JwtVerifier.java b/security-config/src/main/java/com/networknt/security/JwtVerifier.java
@@ -740,6 +740,7 @@ private JsonWebKey getJsonWebKeySetForToken(String kid, Object requestPathOrJwkS
                 for(String serviceId: jwkServiceIds) {
                     config = getJwkConfig(clientConfig, serviceId);
                     jwk = retrieveJwk(kid, config);
+                    if(jwk != null) break;
                 }
             } else {
                 throw new ConfigException("requestPathOrJwkServiceIds must be a string or a list of strings");

Original file line number	Diff line number	Diff line change
`@@ -740,6 +740,7 @@ private JsonWebKey getJsonWebKeySetForToken(String kid, Object requestPathOrJwkS`
`740`	`740`	`for(String serviceId: jwkServiceIds) {`
`741`	`741`	`config = getJwkConfig(clientConfig, serviceId);`
`742`	`742`	`jwk = retrieveJwk(kid, config);`
	`743`	`+ if(jwk != null) break;`
`743`	`744`	`}`
`744`	`745`	`} else {`
`745`	`746`	`throw new ConfigException("requestPathOrJwkServiceIds must be a string or a list of strings");`