Skip to content

Support to use the auth_required decorator on app-wide before_request functions#34

Merged
greyli merged 3 commits intomasterfrom
before-request
Apr 27, 2021
Merged

Support to use the auth_required decorator on app-wide before_request functions#34
greyli merged 3 commits intomasterfrom
before-request

Conversation

@greyli
Copy link
Copy Markdown
Member

@greyli greyli commented Apr 27, 2021

Fix for Flask 2.0.

Checklist:

  • Add tests that demonstrate the correct behavior of the change. Tests should fail without the change.
  • Add an entry in CHANGES.md summarizing the change and linking to the issue and your username.
  • Run pytest and tox, no tests failed.

@codecov-commenter
Copy link
Copy Markdown

codecov-commenter commented Apr 27, 2021

Codecov Report

Merging #34 (6b7d46d) into master (9022c39) will not change coverage.
The diff coverage is 100.00%.

Impacted file tree graph

@@            Coverage Diff            @@
##            master       #34   +/-   ##
=========================================
  Coverage   100.00%   100.00%           
=========================================
  Files           14        14           
  Lines          789       791    +2     
  Branches       146       147    +1     
=========================================
+ Hits           789       791    +2     
Impacted Files Coverage Δ
apiflask/app.py 100.00% <100.00%> (ø)

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 9022c39...6b7d46d. Read the comment docs.

@greyli greyli merged commit ec8c600 into master Apr 27, 2021
@greyli greyli deleted the before-request branch April 27, 2021 13:23
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.

2 participants