[GitHub] [arrow] lidavidm commented on a change in pull request #8325: ARROW-10105: [C++][Python][Java][FlightRPC] Allow disabling server validation

[GitBox <gi...@apache.org>]

lidavidm commented on a change in pull request #8325:
URL: https://github.com/apache/arrow/pull/8325#discussion_r499282185



##########
File path: cpp/src/arrow/flight/client.h
##########
@@ -108,10 +108,14 @@ class ARROW_FLIGHT_EXPORT FlightClientOptions {
   /// positive. When enabled, FlightStreamWriter.Write* may yield a
   /// IOError with error detail FlightWriteSizeStatusDetail.
   int64_t write_size_limit_bytes;
+
   /// \brief Generic connection options, passed to the underlying
   ///     transport; interpretation is implementation-dependent.
   std::vector<std::pair<std::string, util::variant<int, std::string>>> generic_options;
 
+  /// \brief Use TLS without validating the server certificate. Use with caution.
+  bool disable_server_verification;
+
   /// \brief Get default options.
   static FlightClientOptions Defaults();

Review comment:
       I believe you're seeing failures since `Defaults` should be updated to set the new flag, else the flag may or may not be initialized.




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

For queries about this service, please contact Infrastructure at:
users@infra.apache.org