Skip to content

Fix Flask 2x Support - #233

Merged
sloria merged 2 commits into
jmcarp:masterfrom
KyleJamesWalker:fix-send-static-file-lambda-flask-v2
Mar 30, 2022
Merged

Fix Flask 2x Support#233
sloria merged 2 commits into
jmcarp:masterfrom
KyleJamesWalker:fix-send-static-file-lambda-flask-v2

Conversation

@KyleJamesWalker

@KyleJamesWalker KyleJamesWalker commented Mar 23, 2022

Copy link
Copy Markdown
Contributor

When using Flask 2x the send_static_file function is a lambda and
crashes the extraction of the rules_by_endpoint function. There is
currently a PR that has a fix but it's been stale so I'm adding this as
a possible alternate fix.

The other fix is: #230

Closes #229

Note the unit tests were failing before this PR, they are now working again.

If anyone wants to use this version while we wait to have it merged and released I was able to add the following to my setup.py:

"flask-apispec @ https://github.com/KyleJamesWalker/flask-apispec/tarball/fix-send-static-file-lambda-flask-v2#egg=flask-apispec-0.11.0"

When using Flask 2x the send_static_file function is a lambda and
crashes the extraction of the rules_by_endpoint function. There is
currently a PR that has a fix but it's been stale so I'm adding this as
a possible alternate fix.

The other fix is: jmcarp#230
@KyleJamesWalker KyleJamesWalker changed the title Skip static rules Fix Flask 2x Support Mar 23, 2022
@KyleJamesWalker

Copy link
Copy Markdown
Contributor Author

@sloria it looks like you also might be a maintainer of this repo.

@sloria

sloria commented Mar 29, 2022

Copy link
Copy Markdown
Collaborator

sorry for the delay; i'm not super active on this project. i'll merge and release this when i have some time off the clock.

@KyleJamesWalker

Copy link
Copy Markdown
Contributor Author

No worries, thank you!

@sloria
sloria merged commit 74fb204 into jmcarp:master Mar 30, 2022
jollysahil pushed a commit to levrofin/flask-apispec that referenced this pull request Sep 12, 2023
* Skip static rules

When using Flask 2x the send_static_file function is a lambda and
crashes the extraction of the rules_by_endpoint function. There is
currently a PR that has a fix but it's been stale so I'm adding this as
a possible alternate fix.

The other fix is: jmcarp#230

* Update changelog

Co-authored-by: Steven Loria <sloria1@gmail.com>
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.

KeyError: '<lambda>' raises using the method 'register_existing_resources'

2 participants