Skip to content

Conversation

CristianCantoro
Copy link
Contributor

As per subject.
Tested with ~ 5kB request was return an error (request too large)

@CristianCantoro
Copy link
Contributor Author

Rewrote history, removed the else: pass part.

@valhallasw
Copy link
Owner

I still think it would be better to have

if isinstance(v, str):
    v = v.decode('utf-8')

...

(k, v.encode('utf-8'), ...)

as we tell the server the data will be utf-8. I think it's cleaner to actually check this, instead of just passing the str (which could also contain, say, latin-1 data).

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants