michaelkoepf commented on PR #1245: URL: https://github.com/apache/fluss/pull/1245#issuecomment-3591278083
> fail-fast by throwing an exception at the configuration stage, rather than passing it on and relying on dependency's behaviour i think this was considered in the beginning. the problem there was that deactivating the token delegation is a server side config. on the client, we always prepend the token delegation provider to the chain and at runtime, a valid provider will be figured out automatically. in general, you could also introduce another client config that indicates if the token delegation is expected to be active, but this is also not very user friendly. bottom line: i think the easiest way is to just leave default providers in place. and we put a warning in the docs that short-term credentials should be preferred if possible. wdyt? @leekeiabstraction -- 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]
