cccs-cat001 commented on PR #3170: URL: https://github.com/apache/polaris/pull/3170#issuecomment-3607320032
> I find the changes in this PR a bit invasive tbh. I would rather explore Quarkus OIDC token propagation techniques, and more specifically, I would try to inject `io.quarkus.oidc.client.Tokens` wherever necessary: > > https://quarkus.io/guides/security-openid-connect-client-reference#inject-tokens > > But I don't want to delay this PR that is extremely useful. I think we can go with the approach taken here and then switch to token propagation in a follow-up PR. that looks pretty simple compared to all the changes I'm making tbh. Is everything set up so that I can just `@Inject Token token;`? 🤔 -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific comment. To unsubscribe, e-mail: [email protected] For queries about this service, please contact Infrastructure at: [email protected]
