Fix type of aiohttp raw headers

[rsmeral]

Description

aiohttp's ClientResponse.raw_headers is supposed to be "unconverted bytes".

Pook was setting the keys and values of raw_headers as string instead of bytes.

This was causing issues in clients which expect the headers to be bytes, for example aiobotocore tries to decode() header names and values which then fails with AttributeError: 'str' object has no attribute 'decode'.
See https://github.com/aio-libs/aiobotocore/blob/master/aiobotocore/endpoint.py#L41-L56

PR Checklist

• I've added tests for any code changes
• I've documented any new features

[sarayourfriend]

Thanks again @rsmeral. I'll review this later today or sometime tomorrow 🙂

[rsmeral]

Hi @sarayourfriend, the tests seem to be passing. Anything else I can do to help get it merged, so that we can get it out in the next release? Thanks!

[sarayourfriend]

Sorry for the delay, @rsmeral. Things got busy in life and this fell off my radar. I've merged and will release this fix along with another later today.

[rsmeral]

Thanks, and no worries at all!

[rsmeral]

Hi @sarayourfriend, do you perhaps have a rough ETA for the release? Not pushing – just want to adjust our plans. Thanks!

[sarayourfriend]

@rsmeral finally got around to this, released as v2.1.4: https://github.com/h2non/pook/releases/tag/v2.1.4

I mention in the release notes there, but I'm very sorry for the delay in this release. I've had a spate of things that keep coming up over the last 6 months (including most recently a broken hand) that have made this difficult to get around to. As mentioned in the notes, I will have some time to look into ways to improve the release process for pook so this doesn't happen again in the future.

Thanks again as always for your contributions to the project 🙏