- some pep8 problems
- some confusion with old and new code after merging
from old keymanager

Twisted 16.5 sends a BytesIO as message, which was unexpected in this
types list.

this looks like a careless migration to twisted failures.
who knows how long this was like this, this should be covered by tests.

Following Twisted documentation, I found out that the warning method
doesn't exist on logger class. That class has warn method instead. And I
got a error because of that.

https://twistedmatrix.com/documents/16.4.1/api/twisted.logger.Logger.html

Before we used to have content_doc.content_type = 'text/plain;
charset="utf-8"'.
Currently, we have content_doc.content_type = 'text/plain'.

This change will give us, on top of that: content_doc.charset = 'utf-8'.
One can reconstruct the original one using .content_type and .charset.