adutra commented on PR #3170: URL: https://github.com/apache/polaris/pull/3170#issuecomment-3606775602
> @adutra : could you double check quarkus auth integration changes (they LGTM) 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. -- 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]
