Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

select jwk by kid #458

Open
wants to merge 2 commits into
base: main
Choose a base branch
from

Conversation

nathanloyer
Copy link
Contributor

I found that it was useful for my deployment to have multiple JWKS configured for the same algorithm. So I thought to add an optional input parameter for the bulk export tests to allow us to select which one to use via kid.

Let me know if you think this is/isn't generally useful for the community. We can continue to use our fork of the project if you don't want to merge this, but I'd prefer to get it merged. In my local testing I found this to behave identically if you do not set the kid input. If you do set it, then it is picking the correct one.

Let me know if there are automated test cases that should be updated or if you have any other suggestions.

Thanks

@nathanloyer nathanloyer force-pushed the feature/select-jwk-by-kid branch from dca21b6 to da4e6e7 Compare September 29, 2023 18:51
@nathanloyer
Copy link
Contributor Author

Hello @Jammjammjamm,

By all means there's no rush on this, but at some point could you or someone else review this and consider whether this is a feature worth merging? We can always continue to build our server from a fork if this doesn't seem generally useful to everyone.

For more context on why we did this, I wanted to test out different methods and encryption settings for generating JWKS for various test clients. If we can't pick which kid to use then we would have to have separate inferno deployments to test each.

@Jammjammjamm
Copy link
Contributor

We are currently focused on the SVAP update, but this is on our radar.

@Jammjammjamm
Copy link
Contributor

@nathanloyer I wanted to provide you with an update since this has been open for a while. Currently we only have bulk data tests here inside of the g10 test kit, but we are in the process of extracting those into their own test kit, similar to how we have standalone US Core and SMART App Launch tests. We are planning to incorporate the functionality in this PR as part of that work.

@nathanloyer
Copy link
Contributor Author

Thanks @Jammjammjamm. Is there a repo for that new test kit yet?

@nathanloyer
Copy link
Contributor Author

I see the kit support was merged in to the smart app launch test kit. I'll give that a try soon. Thanks

@Jammjammjamm
Copy link
Contributor

@nathanloyer We've just released a standalone bulk data test kit that incorporates this functionality: https://github.com/inferno-framework/bulk-data-test-kit

I'm not sure what the timeline will be for integrating this back into the g10 tests.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants