keyfunc icon indicating copy to clipboard operation
keyfunc copied to clipboard

Allow `SkipAll` option for JWK validation in `keyfunc.NewDefault`

Open d-g-town opened this issue 1 year ago • 1 comments

Following up from this issue thread: https://github.com/MicahParks/jwkset/issues/32

Current working around this by creating storage with jwkset.NewMemoryStorage(), adding the key manually, and specifying the storage in the keyfunc.New(options).

d-g-town avatar May 27 '24 20:05 d-g-town

Thank you for opening this issue @d-g-town. I plan on exposing github.com/MicahParks/jwkset options via a new function in an upcoming release. I'll use this issue to track that work. This is unlikely to happen in the next week or two.

MicahParks avatar May 30 '24 01:05 MicahParks

Wow, almost a year later and I just started the issue. It seems life only gets more busy as time goes on.

I have a PR for the initial implementation to resolve this issue. It adds a new function NewDefaultOverrideCtx, which is the same as NewDefaultCtx, but overrides specific behaviors.

@d-g-town, @kevmo314, @aivajoe, would any of you like to help review this PR?

https://github.com/MicahParks/keyfunc/pull/137

MicahParks avatar May 24 '25 17:05 MicahParks

@d-g-town please see the newly released v3.4.0.

MicahParks avatar May 28 '25 12:05 MicahParks