CGI.escape emoji name

[jayroh]

The commonly used +1, when located somewhere like S3, will not work
as the + is not escaped, causing the browser to think it's a space
instead of a literal +. To replicate, upload +1.png to S3 and try to
bring it up in Chrome.

This commit escapes the emoji name before returning the img tag.

[jch]

Sounds reasonable. Could you take a look at the build failures?

[jayroh]

Hey Jerry - Yeah I'm looking into them right now. Seems that the CGI isn't
being required up on Travis but, oddly enough, pass locally. I'll get it
situated today for final review.

On Wed, Nov 20, 2013 at 10:23 AM, Jerry Cheung [email protected]:

Sounds reasonable. Could you take a look at the build failures?

—
Reply to this email directly or view it on GitHubhttps://github.com//pull/92#issuecomment-28897685
.

[jayroh]

Ha - hadn't taken the older ruby versions into consideration, so that's why.

@jch we're back to green across the board at Travis. Also squashed and force pushed my branch.

[jch]

👍 awesome, thanks for the pull! Release coming soon.

[jayroh]

🎉 Wooo! Thanks :)

On Wed, Nov 20, 2013 at 1:28 PM, Jerry Cheung [email protected]:

[image: 👍] awesome, thanks for the pull! Release coming soon.

—
Reply to this email directly or view it on GitHubhttps://github.com//pull/92#issuecomment-28915009
.