search

etianen / aiohttp-wsgi

WSGI adapter for aiohttp.
https://aiohttp-wsgi.readthedocs.io
BSD 3-Clause "New" or "Revised" License
232 stars 20 forks source link

Fix test to work with aiohttp 3.8.1+ #33

Closed u8sand closed 2 years ago

u8sand commented 2 years ago

Fixes #32

I'm not sure this is a bug but rather a sane modification to aiohttp to add default Content-Type headers as application/octet-stream. Throughout the tests in both aiohttp server & client it's asserted to be application/octet-stream when unset. This change was made upstream here.

I've opted to patch the tests here by adopting this convention of using application/octet-stream but also specifying it explicitly so that it still works with older versions.

etianen commented 2 years ago

Thanks! Some other lints were failing, but your patch looks good. :)