sysid / sse-starlette

BSD 3-Clause "New" or "Revised" License
505 stars 36 forks source link

fix charset #29

Closed synodriver closed 2 years ago

synodriver commented 2 years ago

Current implementation ignores the charset field in the response class, which may lead to some problem, while starlette handles that properly

So we can reuse starlette's code to set proper content-type and charset, like changing this line.

synodriver commented 2 years ago

image

With this patch, the proper charset is set like the base class in starlette

sysid commented 2 years ago

Thanks @synodriver, will merge.