(arrow-rs) branch master updated: fix: enable TLS roots for flight CLI client (#6640)

[tustvold]

This is an automated email from the ASF dual-hosted git repository.

tustvold pushed a commit to branch master
in repository https://gitbox.apache.org/repos/asf/arrow-rs.git


The following commit(s) were added to refs/heads/master by this push:
     new 2983dc124f fix: enable TLS roots for flight CLI client (#6640)
2983dc124f is described below

commit 2983dc124f4472859cc328d0652a872d7e67248b
Author: Marco Neumann <ma...@crepererum.net>
AuthorDate: Tue Oct 29 16:03:00 2024 +0100

    fix: enable TLS roots for flight CLI client (#6640)
    
    Otherwise you get:
    
    ```text
    Error: setup client
    
    Caused by:
        0: connect to endpoint
        1: transport error
        2: invalid peer certificate: UnknownIssuer
        3: invalid peer certificate: UnknownIssuer
    ```
    
    Also see https://github.com/hyperium/tonic/issues/1904 .
---
 arrow-flight/src/bin/flight_sql_client.rs | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/arrow-flight/src/bin/flight_sql_client.rs 
b/arrow-flight/src/bin/flight_sql_client.rs
index 8f0618f495..7b9e34898a 100644
--- a/arrow-flight/src/bin/flight_sql_client.rs
+++ b/arrow-flight/src/bin/flight_sql_client.rs
@@ -357,7 +357,7 @@ async fn setup_client(args: ClientArgs) -> 
Result<FlightSqlServiceClient<Channel
         .keep_alive_while_idle(true);
 
     if args.tls {
-        let tls_config = ClientTlsConfig::new();
+        let tls_config = ClientTlsConfig::new().with_enabled_roots();
         endpoint = endpoint
             .tls_config(tls_config)
             .context("create TLS endpoint")?;