Zappa icon indicating copy to clipboard operation
Zappa copied to clipboard

Handle spaces in x-forwared-for/remove six (#1127)

Open monkut opened this issue 3 years ago • 1 comments

  • Whitespace in X-Forwarded-For is optional and therefore should handle the case when there is no whitespace (contribution by naresh-uplight)
  • :fire: remove six from zappa dependencies
  • :wrench: Fix url decoding for query string (contribution by choich)

six is still a sub-dependency of boto3, but no longer required directly by zappa.

Related Issue(s): https://github.com/zappa/Zappa/issues/879 https://github.com/zappa/Zappa/issues/1127 https://github.com/zappa/Zappa/issues/1164

Related PR: https://github.com/zappa/Zappa/pull/972 (integrated into this PR) contribution by (pauldes)

monkut avatar Aug 10 '22 07:08 monkut

Coverage Status

Coverage increased (+0.1%) to 73.81% when pulling e78cd9c58c177928332545fad036b19bfe845a79 on fix/issue-1127-handle-optional-space-in-xforwardedfor-header into c048ea20254f549b2a68e66a22bd548946570824 on master.

coveralls avatar Aug 10 '22 07:08 coveralls