Skip to content

Commit

Permalink
✨ Update ORJSONResponse to support non str keys and serializing N…
Browse files Browse the repository at this point in the history
…umpy arrays (#3892)

Co-authored-by: Sebastián Ramírez <tiangolo@gmail.com>
Co-authored-by: pre-commit-ci[bot] <66853113+pre-commit-ci[bot]@users.noreply.github.com>
  • Loading branch information
3 people committed Sep 2, 2022
1 parent 173b891 commit 7250c19
Show file tree
Hide file tree
Showing 2 changed files with 24 additions and 1 deletion.
4 changes: 3 additions & 1 deletion fastapi/responses.py
Expand Up @@ -31,4 +31,6 @@ 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 | orjson.OPT_SERIALIZE_NUMPY
)
21 changes: 21 additions & 0 deletions tests/test_orjson_response_class.py
@@ -0,0 +1,21 @@
from fastapi import FastAPI
from fastapi.responses import ORJSONResponse
from fastapi.testclient import TestClient
from sqlalchemy.sql.elements import quoted_name

app = FastAPI(default_response_class=ORJSONResponse)


@app.get("/orjson_non_str_keys")
def get_orjson_non_str_keys():
key = quoted_name(value="msg", quote=False)
return {key: "Hello World", 1: 1}


client = TestClient(app)


def test_orjson_non_str_keys():
with client:
response = client.get("/orjson_non_str_keys")
assert response.json() == {"msg": "Hello World", "1": 1}

0 comments on commit 7250c19

Please sign in to comment.