(airflow) branch main updated: Add connexion<3.0 upper bound (#35218)

[po...@apache.org]

This is an automated email from the ASF dual-hosted git repository.

potiuk pushed a commit to branch main
in repository https://gitbox.apache.org/repos/asf/airflow.git


The following commit(s) were added to refs/heads/main by this push:
     new f592b2c40b Add connexion<3.0 upper bound (#35218)
f592b2c40b is described below

commit f592b2c40bd23a7bb1e3b471e47c5d1968f70a45
Author: Kevin C <Ka...@users.noreply.github.com>
AuthorDate: Sat Oct 28 12:23:52 2023 -0700

    Add connexion<3.0 upper bound (#35218)
    
    * Add connexion<3.0 upper bound
    
    otherwise, breakage at import: airflow/www/extensions/init_views.py
    
    Fix: apache/airflow/issues/35217
    
    * Update setup.cfg
    
    * Update setup.cfg
    
    ---------
    
    Co-authored-by: Jarek Potiuk <ja...@potiuk.com>
---
 setup.cfg | 8 +++++++-
 1 file changed, 7 insertions(+), 1 deletion(-)

diff --git a/setup.cfg b/setup.cfg
index c63db90490..21115ce83e 100644
--- a/setup.cfg
+++ b/setup.cfg
@@ -79,7 +79,13 @@ install_requires =
     # Update CustomTTYColoredFormatter to remove
     colorlog>=4.0.2, <5.0
     configupdater>=3.1.1
-    connexion[flask]>=2.10.0
+    # `airflow/www/extensions/init_views` imports `connexion.decorators.validation.RequestBodyValidator`
+    # connexion v3 has refactored the entire module to middleware, see: /spec-first/connexion/issues/1525
+    # Specifically, RequestBodyValidator was removed in: /spec-first/connexion/pull/1595
+    # The usage was added in #30596, seemingly only to override and improve the default error message.
+    # Either revert that change or find another way, preferably without using connexion internals.
+    # This limit can be removed after https://github.com/apache/airflow/issues/35234 is fixed
+    connexion[flask]>=2.10.0,<3.0
     cron-descriptor>=1.2.24
     croniter>=0.3.17
     cryptography>=0.9.3