flask-mwoauth icon indicating copy to clipboard operation
flask-mwoauth copied to clipboard

Fix encoding issue, lower the limit for multipart requests

Open CristianCantoro opened this issue 11 years ago • 2 comments

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

CristianCantoro avatar Feb 10 '14 20:02 CristianCantoro

Rewrote history, removed the else: pass part.

CristianCantoro avatar Feb 14 '14 16:02 CristianCantoro

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).

valhallasw avatar Feb 14 '14 19:02 valhallasw