You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@arrow.apache.org by tu...@apache.org on 2023/05/24 17:14:36 UTC

[arrow-rs] branch master updated: Strip leading whitespace from flight_sql_client custom header values (#4271)

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 136445f5e Strip leading whitespace from flight_sql_client custom header values (#4271)
136445f5e is described below

commit 136445f5e63501e26e6be21e68a56a5f8e8b1e2e
Author: Marko Mikulicic <mk...@influxdata.com>
AuthorDate: Wed May 24 19:14:30 2023 +0200

    Strip leading whitespace from flight_sql_client custom header values (#4271)
    
    * Strip leading whitespace from flight_sql_client custom header values
    
    * Also strip trailing whitespace from flight_sql_client custom header values
---
 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 a787989bf..e5aacc2e7 100644
--- a/arrow-flight/src/bin/flight_sql_client.rs
+++ b/arrow-flight/src/bin/flight_sql_client.rs
@@ -49,7 +49,7 @@ where
         match parts.as_slice() {
             [key, value] => {
                 let key = K::from_str(key).map_err(|e| e.to_string())?;
-                let value = V::from_str(value).map_err(|e| e.to_string())?;
+                let value = V::from_str(value.trim()).map_err(|e| e.to_string())?;
                 Ok(Self { key, value })
             }
             _ => Err(format!(