StreetVoice / django-cloudfront

CloudFront private content signing for Django
MIT License
11 stars 2 forks source link

Escaping issues #1

Open rypalmer opened 11 years ago

rypalmer commented 11 years ago

Thanks for contributing this. The one issue I ran into was the escaping of tildes -- apparently they shouldn't be escaped. I got around this by the following:

cloudfront.sign(resource, 3600).replace('%7E', '~')

This may or may not be related: https://github.com/simplegeo/python-oauth2/issues/15

Also: when outputting the resulting context variable in a template, I had to use |safe to prevent the escaping of ampersands.

tzangms commented 11 years ago

@rypalmer Could you provide me the example url which you encountered the problem?

rypalmer commented 11 years ago

Any URL query parameter containing a tilde I guess. Not all would end up having one. I can't seem to generate with one at the moment.. sorry.

Ryan

On Mon, May 13, 2013 at 6:34 AM, tzangms notifications@github.com wrote:

@rypalmer https://github.com/rypalmer Could you provide me the example url which you encountered the problem?

— Reply to this email directly or view it on GitHubhttps://github.com/StreetVoice/django-cloudfront/issues/1#issuecomment-17801588 .