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.

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

Skip middleware code when there are no user middlewares installed #2629

Merged
merged 5 commits into from
Dec 28, 2017
Merged

Skip middleware code when there are no user middlewares installed #2629

merged 5 commits into from
Dec 28, 2017

Commits on Dec 28, 2017

  1. Skip middleware code when there are no user middlewares installed 

    Fixes performance issue introduced by #2577, boosting from 8K req/sec
to almost 9K req/sec.

If there are no middlewares installed by the user the attribute
`request._match_info.current_app` already looks at to the right app,
this is by design.
    @pfreixes
    pfreixes committed Dec 28, 2017
    Configuration menu
    Copy the full SHA
    bb4a60c View commit details
    Browse the repository at this point in the history

  2. Added file change

    @pfreixes
    pfreixes committed Dec 28, 2017
    Configuration menu
    Copy the full SHA
    54bec92 View commit details
    Browse the repository at this point in the history

  3. Flake8 issues

    @pfreixes
    pfreixes committed Dec 28, 2017
    Configuration menu
    Copy the full SHA
    d13a03a View commit details
    Browse the repository at this point in the history

  4. Make code less cryptic, added more documentation

    @pfreixes
    pfreixes committed Dec 28, 2017
    Configuration menu
    Copy the full SHA
    ae5c137 View commit details
    Browse the repository at this point in the history

  5. Merge branch 'master' into fix_degradation_performance_middleware

    @asvetlov
    asvetlov committed Dec 28, 2017
    Configuration menu
    Copy the full SHA
    7d7cf7f View commit details
    Browse the repository at this point in the history