amoghrajesh commented on code in PR #37214:
URL: https://github.com/apache/airflow/pull/37214#discussion_r1481643804


##########
airflow/providers/mongo/hooks/mongo.py:
##########
@@ -75,6 +74,10 @@ def __init__(self, mongo_conn_id: str = default_conn_name, 
*args, **kwargs) -> N
         self.client: MongoClient | None = None
         self.uri = self._create_uri()
 
+        self.allow_insecure = True

Review Comment:
   Didn't completely get you here. The user can always come back to old 
behaviour by setting the ssl as False, right or not specifying it at all. 
   
   The old behaviour was that irrespective of ssl, we skipped tls validation. 
Should we retain that behaviour?



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