Ensure handler patterns are str, not unicode #1368
Merged
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Python < 2.6.5 doesn't accept unicode keys in
f(**kwargs)
, and base_project_url will always be unicode, which will in turn make the patterns unicode, and ultimately result in unicode keys in kwargs tohandler._execute(**kwargs)
in tornado (by way ofre.match.groupdict()
). This enforces that base_project_url be ascii, which it always should be anyway.Note that the URLs these patterns check against are escaped, and thus guaranteed to be ASCII: 'héllo' is really 'h%C3%A9llo'.
If you actually use u'héllo' in your regex, it will not match the URLs you think it should.
Bug introduced by PR #1332
closes #1360