omniauth / omniauth_openid_connect

MIT License
168 stars 187 forks source link

Add `parse_jwk_key_from_hash` to handle jwks Hash #158

Closed takahashim closed 1 year ago

takahashim commented 1 year ago

Fix #156

parse_jwk_key_from_hash() has been added to handle the Hash returned by ::OpenIDConnect.http_client.get(client_options.jwks_uri).body. The implementation of parse_jwk_key() has also been updated to use parse_jwk_key_from_hash().

stanhu commented 1 year ago

Thanks, closing in favor of #157.

davidwessman commented 1 year ago

@takahashim Well done with the initial update and making a PR for a fix as well 🙂