Skip to content
This repository has been archived by the owner on May 31, 2022. It is now read-only.

Allow JwkTokenStore to be created with multiple JWK Set URLs #1050

Closed
habuma opened this issue Apr 25, 2017 · 6 comments
Closed

Allow JwkTokenStore to be created with multiple JWK Set URLs #1050

habuma opened this issue Apr 25, 2017 · 6 comments

Comments

@habuma
Copy link

habuma commented Apr 25, 2017

As it stands, JwkTokenStore only supports a single JWK Set URL. In SCS, we have potential for multiple token issuers (various UAA identity zones), and therefore multiple JWK Set URLs.

JwkTokenStore should either support multiple JWK Set URLs or there should be a separate implementation (perhaps MultipleIssuerJwkTokenStore) that does support multiple issuers and JWK Set URLs.

@rwinch rwinch added this to the 2.2.0.M1 milestone Jun 28, 2017
@rwinch rwinch self-assigned this Jun 28, 2017
@wojtekk
Copy link

wojtekk commented Aug 7, 2017

Great job guys!

Can I use this feature based on a configuration in property file only?

If this is possible, I can't figure out how to that - could you provide any example?

@jgrandja
Copy link
Contributor

@wojtekk Yes, the Spring boot property is security.oauth2.resource.jwk.key-set-uri. I noticed that it's not documented in the Spring Boot reference so I logged this issue.

@jgrandja
Copy link
Contributor

jgrandja commented Aug 16, 2017

@wojtekk Also, keep in mind that this property supports only one JwkSet uri. If you want to specify multiple uri's than you will need to log a new issue in the Spring Boot repo to add this capability.

@wojtekk
Copy link

wojtekk commented Aug 16, 2017

@jgrandja thanks, I was thinking about multiple properties - I will do that (or maybe I will try to prepare PR)

@KishorBachhav
Copy link

I can see that support for multiple JWK Set URLs is added but how to specify multiple urls through properties file.
@wojtekk did you get chance to file a separate issue so that I can refer to the same rather than me filing duplicate.

@KishorBachhav
Copy link

Issue#1479 is filed to provide support to specify multiple urls through properties file

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Development

No branches or pull requests

5 participants