istvan-fodor commented on issue #5662: URL: https://github.com/apache/arrow-rs/issues/5662#issuecomment-2064639084
Update on the first issue (see [discussion](https://lists.apache.org/thread/66zd5k3l63vrztjjmgzdh3xm389cbkbk)). It looks like that the expectation would be to set the authorization header on the response. I took a look at the `FlightSqlServiceClient` and it looks like the authorization header is indeed expected in the handshake response: ```rust let resp = self .flight_client .handshake(req) .await .map_err(|e| ArrowError::IpcError(format!("Can't handshake {e}")))?; if let Some(auth) = resp.metadata().get("authorization") { let auth = auth .to_str() .map_err(|_| ArrowError::ParseError("Can't read auth header".to_string()))?; let bearer = "Bearer "; if !auth.starts_with(bearer) { Err(ArrowError::ParseError("Invalid auth header!".to_string()))?; } let auth = auth[bearer.len()..].to_string(); self.token = Some(auth); } ``` -- 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]
