-
Notifications
You must be signed in to change notification settings - Fork 309
unicode flub in GitHub API usage #1489
Comments
Example: https://www.gittip.com/on/github/atdt%E2%80%8E/ Obtained from the people search box by filling in the input box 'atdt' + something else I have no clue how I inserted it :/ |
Could be fixed by #158. I remember going over the code yesterday and thinking it would blow with unicode 😄 |
Sounds right to me. |
How did this ever occur? We don't allow unicode in usernames, and neither does Github |
You can put anything in the url https://www.gittip.com/on/github/[username]/ and we are not sanitizing that. |
Perhaps it was a security researcher? |
@seanlinsley I just somehow entered a search for "atdt" and eventually that led to "atdt%E2%80%8E" because I added some additional unicode after my "atdt" search sentence. The server informed me of some error so I have pinged folks on irc and @whit537 grabbed the server side stack trace. |
This problem has been solved, probably by #1369. |
https://www.gittip.com/on/github/atdt%E2%80%8E/ gives a nice error page now. Thanks! |
Or even just in logging?
https://app.getsentry.com/gittip/gittip/group/6850474/
The text was updated successfully, but these errors were encountered: