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

✨ Update ORJSONResponse to support non str keys and serializing Numpy arrays #3892

Merged
merged 8 commits into from Sep 2, 2022
Merged

✨ Update ORJSONResponse to support non str keys and serializing Numpy arrays #3892

Changes from 1 commit
Commits
Show all changes
8 commits
Select commit Hold shift + click to select a range
d81ea3a
Update orjson support non str keys
baby5 Sep 11, 2021
09cd404
Update add orjson non_str_keys test
baby5 Oct 6, 2021
6d45302
Update add integer case
baby5 Oct 6, 2021
49a9c31
Merge branch 'master' into master
tiangolo Sep 2, 2022
48c6165
🔧 Update ORJSONResponse to serialize Numpy by default
tiangolo Sep 2, 2022
37621c8
🎨 [pre-commit.ci] Auto format from pre-commit.com hooks
pre-commit-ci[bot] Sep 2, 2022
83d3c66
⏪ Revert test in custom responses that doesn't use the changes in ORJ…
tiangolo Sep 2, 2022
f136624
✅ Add test for changes in ORJSONResponse
tiangolo Sep 2, 2022
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Jump to
Jump to file
Failed to load files.
Diff view
Diff view
2 changes: 1 addition & 1 deletion fastapi/responses.py
View file
Open in desktop
Expand Up @@ -31,4 +31,4 @@ class ORJSONResponse(JSONResponse):

def render(self, content: Any) -> bytes:
assert orjson is not None, "orjson must be installed to use ORJSONResponse"
return orjson.dumps(content)
return orjson.dumps(content, option=orjson.OPT_NON_STR_KEYS)