Jefffrey commented on code in PR #5666:
URL: https://github.com/apache/arrow-rs/pull/5666#discussion_r1575839969


##########
arrow-flight/examples/flight_sql_server.rs:
##########
@@ -725,9 +742,8 @@ impl FlightSqlService for FlightSqlServiceImpl {
 /// This example shows how to run a FlightSql server
 #[tokio::main]
 async fn main() -> Result<(), Box<dyn std::error::Error>> {
-    let addr = "0.0.0.0:50051".parse()?;
-
-    let svc = FlightServiceServer::new(FlightSqlServiceImpl {});
+    let addr_str = "127.0.0.1:50051";

Review Comment:
   Ok I figured out what's going on. It's not actually a TLS error, is an error 
with `location` itself.
   
   When `0.0.0.0` is supplied, this means the server is listening on it, but 
also with these changes it serves as the location that is returned as the 
endpoints for any FlightInfo returned by the server. And trying to connect to 
0.0.0.0 isn't going to work. Hence why substituting 127.0.0.1 works, as that 
can be connected to.



-- 
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]

Reply via email to