dennishuo commented on issue #146: URL: https://github.com/apache/polaris/issues/146#issuecomment-2303565592
This was definitely an unfortunate oversight; good catch! Luckily, since the bug wasn't that the request handlers actually expected to parse `true` from the String, just that they only checked `!isNullOrEmpty` to assume it meant vended-credentials, it's forward-compatible if anyone is using the "correct" syntax. We should try to update docs as quickly as possible to reduce the degree to which `X-Iceberg-Access-Delegation: true` gets entrenched into various workflows or test scripts of anyone integrating with Polaris. At this point it may already be prudent to have an option for still falling back to the `vended-credentials` behavior if none of the comma-separated list of supported mechanisms is recognized (including the string "true"). -- 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: dev-unsubscr...@polaris.apache.org For queries about this service, please contact Infrastructure at: us...@infra.apache.org
