8000 Incompatibility with flask .11 error handling: TypeError: 'type' object is not iterable · Issue #58 · flask-api/flask-api · GitHub
[go: up one dir, main page]
Skip to content

Incompat 8000 ibility with flask .11 error handling: TypeError: 'type' object is not iterable #58

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.

Sign up for GitHub

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

Jump to bottom
Closed
vitriol opened this issue Jun 22, 2016 · 3 comments
Closed

Incompatibility with flask .11 error handling: TypeError: 'type' object is not iterable #58

vitriol opened this issue Jun 22, 2016 · 3 comments

Comments

@vitriol
Copy link
vitriol commented Jun 22, 2016

Flask 0.11 Changed the order in which errorhandlers are executed and they added the type of the error handler in there making any app that registers flask-api to throw an error before being able to execute the proper error handler (app.py line 91)

What seems to work for me is to add the following in app.py:91

for typecheck, handler in chain(blueprint_handlers.items(), app_handlers.items()):

since blueprint_handlers and app_handlers are now both dict objects
@jacebrowning
Copy link
Member

A pull request including a test for this fix would be greatly appreciated!

@vitriol
Copy link
Author
vitriol commented Jun 22, 2016

I would appreciate that too! I have never done that before and sadly I don't have time anytime soon to try it

dutronlabs pushed a commit to dutronlabs/flask-api that referenced this issue Jul 18, 2016
fix for flask-api#58, handlers are dict objects
949890f
@dutronlabs dutronlabs mentioned this issue Jul 18, 2016
Merged
jacebrowning added a commit that referenced this issue Jul 20, 2016
@jacebrowning
Merge pull request #59 from dutronlabs/master
5575a62 
fix for #58
@jacebrowning
Copy link
Member

Flask-API ~= 0.6.7 should work.

@jacebrowning jacebrowning mentioned this issue Aug 18, 2016
Closed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
2 participants
@vitriol @jacebrowning
0