Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

requirements.txt -> Werkzeug #764

Closed
drellxor opened this issue May 13, 2021 · 7 comments
Closed

requirements.txt -> Werkzeug #764

drellxor opened this issue May 13, 2021 · 7 comments
Labels
question Further information is requested stale

Comments

@drellxor
Copy link

Question
Hello!
FARM has strictly specified Werkzeug==0.16.1 version in requirements. And it is not really comfortable for integrating your another project haystack into existing flask application, because version 0.16.1 is quite outdated and dependency conflict occurs.

What do you think about removing Werkzeug version number from requirements? I can do it on my own, check if everything works fine and then create pull request. Just want to know, maybe version was strictly specified for some particular reason.

@drellxor drellxor added the question Further information is requested label May 13, 2021
@julian-risch
Copy link
Member

julian-risch commented May 14, 2021

Hi @drellxor The reason why we fixed the version of Werkzeug is that flask-restplus library was broken by an update of Werkzeug (noirbizarre/flask-restplus#777 and also discussed here for example jarus/flask-testing#143). As a quick-fix, we pinned the version of Werkzeug in FARM requirements. Here is the corresponding pull request #250
It would be great if you could check whether this problem has been fixed and whether a more recent version of Werkzeug works with flask-restplus!

@drellxor
Copy link
Author

@julian-risch Thank you for you answer! As I know, flask-restplus is no longer maintained and I'm afraid there is no way it will be fixed there. But there's a great fork flask-restx and it's official as I know. They fixed problem with Werkzeug there (issue: python-restx/flask-restx#34, pull request: python-restx/flask-restx#35)

What do you think about changing from flask-restplus to flask-restx?

@julian-risch
Copy link
Member

You are right. flask-restx is a better alternative. 👍 I would appreciate changing from flask-restplus to flask-restx and upgrading Werkzeug.

@julian-risch
Copy link
Member

@drellxor would you like to contribute by making the change, checking if everything works fine and then creating a pull request?

@drellxor
Copy link
Author

@julian-risch Yeah, sure.
But we probably should wait a couple of days until flast-restx 0.4.0 release, because latest flask 2.0.0 release that happened about a week ago breaks current flask-restx release.

Other option may be just to pin flask version to <= 2.0.0. But in this case we may face similar issue a bit later.

Which option do you think will be better?

@julian-risch
Copy link
Member

Great to hear that! I think the first option is better. We can wait until flast-restx 0.4.0 release.

@stale
Copy link

stale bot commented Sep 16, 2021

This issue has been automatically marked as stale because it has not had recent activity. It will be closed in 21 days if no further activity occurs.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
question Further information is requested stale
Projects
None yet
Development

No branches or pull requests

2 participants