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

Pin httpbin 0.7.0 and werkzeug 2.0.3 #353

Merged
merged 1 commit into from
Aug 21, 2022
Merged

Conversation

twm
Copy link
Contributor

@twm twm commented Aug 14, 2022

Fixes #352.

httpbin 0.7.0 gets us compatibility with current versions of MarkupSafe, which dropped soft_unicode in version 2.1.0.

werkzeug 2.0.3 is the most recent version that includes the deprecated BaseResponse symbol that httpbin imports.

This is just a quick fix. To properly address this we'll have to address the fact that httpbin hasn't released since 2018. This also doesn't address the very lax pinning of test dependencies in general.

httpbin 0.7.0 gets us compatibility with current versions of MarkupSafe,
which dropped `soft_unicode` in version 2.1.0 [1].

werkzeug 2.0.3 [2] is the most recent version that includes the deprecated
BaseResponse symbol that httpbin imports.

[1]: https://markupsafe.palletsprojects.com/en/2.1.x/changes/#version-2-1-0
[2]: https://werkzeug.palletsprojects.com/en/2.2.x/changes/#version-2-0-3
@twm twm requested a review from a team August 14, 2022 01:22
Copy link
Member

@adiroiban adiroiban left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM. Thanks!

@twm twm merged commit 54ec473 into trunk Aug 21, 2022
@twm twm deleted the 352-werkzeug-import-error branch August 21, 2022 20:16
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Tests are broken due to Werkzeug ImportError
2 participants