-
Notifications
You must be signed in to change notification settings - Fork 49
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
Support for data:
URLs for loading resources
#593
Comments
@MikeEdgar Hi Mike, would it really be about supporting arbitrary URL schemes for https://github.com/smallrye/smallrye-jwt/blob/main/implementation/common/src/main/java/io/smallrye/jwt/util/ResourceUtils.java#L81 to work ? But that would require supporting the injection of such resolvers and passing them around in Perhaps, for now, a simpler alternative is to introduce |
if you agree with the |
@sberyozkin I think the suggestion to have |
Hey @MikeEdgar Do you think we can keep this issue as a possible enhancement ? |
@sberyozkin , no objection from me to keep it open. There could be cases where it's useful, although #596 was exactly what I needed at the time 😄 |
Currently, the loading of resources is limited to
file
,https
, andclasspath
URLs [1]. It would be useful to also supportdata
URLs [2] as well. In a cloud environment, this would allow us to directly specify resources via env (from secrets) instead of mounting the secrets as volumes and mapping to files.For example, the configuration could be set to:
Allowing the secret to be mapped to env
MY_PEM_ENCODED_CA_CERT
and loaded by the implementation.[1] https://github.com/smallrye/smallrye-jwt/blob/main/implementation/common/src/main/java/io/smallrye/jwt/util/ResourceUtils.java#L64
[2] https://developer.mozilla.org/en-US/docs/Web/HTTP/Basics_of_HTTP/Data_URIs
The text was updated successfully, but these errors were encountered: