potiuk commented on code in PR #35218:
URL: https://github.com/apache/airflow/pull/35218#discussion_r1375243723
##########
setup.cfg:
##########
@@ -79,7 +79,12 @@ 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.
Review Comment:
I created an issue for that in our repo - to keep track of it. This is what
we do when we limit upper-binding - we create a "Task" issue to fix the problem
when we do - this way we not only see immediately in the code why we limit it,
but we also have an issue to track it which easy easy to track and search.
--
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]
