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

🐛 Fix empty reponse body when default status_code is empty but the a Response parameter with response.status_code is set #5360

Merged
merged 5 commits into from Sep 8, 2022
Merged

🐛 Fix empty reponse body when default status_code is empty but the a Response parameter with response.status_code is set #5360

merged 5 commits into from Sep 8, 2022

Commits on Sep 7, 2022

  1. Fix bug in get_request_handler (fastapi/routing.py) which cleared rep… 

    …onse content because wrong status code variable has been checked
    @tmeckel
    tmeckel committed Sep 7, 2022
    Configuration menu
    Copy the full SHA
    5892a1d View commit details
    Browse the repository at this point in the history

  2. Added tests/test_reponse_set_reponse_code.py

    @tmeckel
    tmeckel committed Sep 7, 2022
    Configuration menu
    Copy the full SHA
    0dbc75c View commit details
    Browse the repository at this point in the history

  3. 🎨 [pre-commit.ci] Auto format from pre-commit.com hooks

    @pre-commit-ci
    pre-commit-ci[bot] committed Sep 7, 2022
    Configuration menu
    Copy the full SHA
    834bb6d View commit details
    Browse the repository at this point in the history

Commits on Sep 8, 2022

  1. Merge branch 'master' into fix/empty-reponse-body

    @tiangolo
    tiangolo committed Sep 8, 2022
    Configuration menu
    Copy the full SHA
    a942d46 View commit details
    Browse the repository at this point in the history

  2. ✅ Tweak and simplify tests, add tests for OpenAPI

    @tiangolo
    tiangolo committed Sep 8, 2022
    Configuration menu
    Copy the full SHA
    9bf2b96 View commit details
    Browse the repository at this point in the history