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

Reply via email to