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

4.2.2 broke Flask 1.x.x #437

Closed
vimalloc-mavenlink opened this issue Jul 6, 2021 · 7 comments
Closed

4.2.2 broke Flask 1.x.x #437

vimalloc-mavenlink opened this issue Jul 6, 2021 · 7 comments

Comments

@vimalloc-mavenlink
Copy link
Contributor

I'll get a quick fix out ASAP. Tomorrow (after releasing the fix) I'll make sure to add a unit test around this to prevent breakages in the future 😞

vimalloc added a commit that referenced this issue Jul 6, 2021
Still need to get a unit test in place so this doesn't happen again.
Refs #437
@vimalloc vimalloc changed the title 4.2.2 (probably) broke flask v1. 4.2.2 broke Flask 1.x.x Jul 6, 2021
vimalloc added a commit that referenced this issue Jul 6, 2021
@vimalloc
Copy link
Owner

vimalloc commented Jul 6, 2021

Fix has been released in 4.2.3. I'll work on better test setup so this doesn't happen again tomorrow 👍

@haqduong
Copy link

I still got this error in 4.2.3, I think you should check for phrase "'Flask' object has no attribute 'ensure_sync'" is contained in e but not only equal.

@StefanVDWeide
Copy link
Contributor

@haqduong can you share more information on how you produced the error? I can't seem to replicate it in 4.2.3 with Flask 1.1.4.

@vimalloc Happy to help you with the test setup to prevent this from happening again if needed since I was the one who introduced the Flask 2.0 support. Has any progress been made yet?

@vimalloc-mavenlink
Copy link
Contributor Author

I might change the logic to something like this, might be less error prone: https://stackoverflow.com/a/7580687/272689

@StefanVDWeide Per the specs, I haven't actually looked much into it yet. i would totally welcome a PR if you want to help dig into it. At a glance I didn't see a way to configure tox with different versions of the same package but I might be missing something.

@StefanVDWeide
Copy link
Contributor

@vimalloc I've gone ahead and took a first stab at both implementing a better version check as per the SO post you shared (seems like a good solution) and adding an extra test environment to tox which uses flask 1.1.4

#440

Let me know what you think!

@haqduong
Copy link

@StefanVDWeide thank you for your reply, I saw it once when call stack is quite complex but I lost the log atm. But I really like your PR, it's much better implementation for the issue.

@StefanVDWeide
Copy link
Contributor

StefanVDWeide commented Aug 2, 2021

@vimalloc I think now that #440 is merged we can close this issue. Thanks for thinking along and the quick actions!

@vimalloc vimalloc closed this as completed Sep 3, 2021
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

No branches or pull requests

4 participants