jedcunningham commented on code in PR #39528:
URL: https://github.com/apache/airflow/pull/39528#discussion_r1595886254


##########
airflow/providers/amazon/provider.yaml:
##########
@@ -128,6 +124,10 @@ additional-extras:
   - name: python3-saml
     dependencies:
       - python3-saml>=1.16.0
+      # XML sec 1.3.14 breaks Amazon's authentication with `lxml & xmlsec 
libxml2 library version mismatch`
+      # We should investigate if we can upgrade to a newer version of lxml and 
xmlsec
+      # Tracked in https://github.com/apache/airflow/issues/39103
+      - xmlsec<1.3.14

Review Comment:
   `xmlsec` is a dependency of `python3-saml`, so it's always been there, and 
likely will always be.
   
   The change 3 weeks ago was just to set an upper bound. This PR just moves it 
into the `python3-saml` extra.



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