crepererum opened a new pull request, #3788:
URL: https://github.com/apache/arrow-rs/pull/3788
# Which issue does this PR close?
\-
# Rationale for this change
Assorted `FlightSqlServiceClient` improvements:
- **TLS config:** Do NOT alter existing method signatures if the TLS feature
is enabled. Features should be purely additive in Rust. Instead use a new
method to pass TLS configs. The config is now passed as `ClientTlsConfig` to
allow more flexibility, e.g. just to use TLS w/o any client certs.
- **token handlng:** Allow the token to be passed in from an external
source. The [auth spec] is super flexibility ("application-defined") and we
cannot derive a way to determine the token in all cases. The current
handshake-based mechanism is OK though. Also make sure the token is used in all
relevant methods.
- **headers:** Allow users to pass in additional headers. This is helpful
for certain applications.
[auth spec]: https://arrow.apache.org/docs/format/Flight.html#authentication
# What changes are included in this PR?
See list above.
# Are there any user-facing changes?
**Breaking** slight changes in the `FlightSqlServiceClient` interface.
--
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]
