dimas-b commented on code in PR #1164: URL: https://github.com/apache/polaris/pull/1164#discussion_r2005798659
########## polaris-core/src/main/java/org/apache/polaris/core/context/CallContext.java: ########## @@ -134,6 +173,8 @@ public Map<String, Object> contextVariables() { Map<String, Object> contextVariables(); + URI getBaseUri(); Review Comment: Does my suggestion from this comment work? https://github.com/apache/polaris/pull/1164#discussion_r1992328462 Ideally, we should get `UriInfo` at the Web (REST API) layer and use it to produce the credntials endpoint URI there. URIs are a REST API concern. The majority of Polaris Core should not be concerned with endpoint URIs. -- 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: issues-unsubscr...@polaris.apache.org For queries about this service, please contact Infrastructure at: us...@infra.apache.org