diff options
author | Brice Figureau <brice-puppet@daysofwonder.com> | 2009-07-16 20:53:36 +0200 |
---|---|---|
committer | James Turnbull <james@lovedthanlost.net> | 2009-07-17 18:16:06 +1000 |
commit | 8f8240763b0a8ab74b5b78eeb2372a2aa7848049 (patch) | |
tree | 166be7400b18beb81560b1824d739bb876ac38f1 /lib/puppet/util/rdoc/code_objects.rb | |
parent | c86d44ed826b99752fd0ee85b2a77eaadd8a57ae (diff) | |
download | puppet-8f8240763b0a8ab74b5b78eeb2372a2aa7848049.tar.gz puppet-8f8240763b0a8ab74b5b78eeb2372a2aa7848049.tar.xz puppet-8f8240763b0a8ab74b5b78eeb2372a2aa7848049.zip |
Fix #2261 - Make sure query string parameters are properly escaped
The problem is that URI.escape by default doesn't escape '+' (and
some other characters). But some web framework (at least webrick)
unescape the query string behind Puppet's back changing all '+'
to spaces corrupting facts containing '+' characters (like base64
encoded values).
The current fix makes sure we use CGI.escape for all query string
parameters. Indirection keys/path are still using URI escaping because
this part of the URI format shouldn't be handled like query string
parameters (otherwise '/' url separators are encoded which changes
the uri path).
Signed-off-by: Brice Figureau <brice-puppet@daysofwonder.com>
Diffstat (limited to 'lib/puppet/util/rdoc/code_objects.rb')
0 files changed, 0 insertions, 0 deletions