1
0
Fork 0
mirror of https://github.com/ruby/ruby.git synced 2022-11-09 12:17:21 -05:00

fix error if the input is mixed Unicode and percent-escapes

Reported by kivikakk (Ashe Connor) with tests and doc fix
Patch based on mame and fix by naruse
[Bug #14586]

git-svn-id: svn+ssh://ci.ruby-lang.org/ruby/trunk@62695 b2dd03c8-39d4-4d8f-98ff-823fe69b080e
This commit is contained in:
naruse 2018-03-08 09:38:01 +00:00
parent 0a2fb188e3
commit 6db869c101
2 changed files with 10 additions and 3 deletions

View file

@ -315,13 +315,13 @@ module URI
# #
# :call-seq: # :call-seq:
# unescape( str ) # unescape( str )
# unescape( str, unsafe ) # unescape( str, escaped )
# #
# == Args # == Args
# #
# +str+:: # +str+::
# String to remove escapes from # String to remove escapes from
# +unsafe+:: # +escaped+::
# Regexp to apply. Defaults to self.regexp[:ESCAPED] # Regexp to apply. Defaults to self.regexp[:ESCAPED]
# #
# == Description # == Description
@ -329,7 +329,7 @@ module URI
# Removes escapes from +str+ # Removes escapes from +str+
# #
def unescape(str, escaped = @regexp[:ESCAPED]) def unescape(str, escaped = @regexp[:ESCAPED])
str.gsub(escaped) { [$&[1, 2].hex].pack('C') }.force_encoding(str.encoding) str.gsub(escaped) { [$&[1, 2]].pack('H2').force_encoding(str.encoding) }
end end
@@to_s = Kernel.instance_method(:to_s) @@to_s = Kernel.instance_method(:to_s)

View file

@ -45,4 +45,11 @@ class URI::TestParser < Test::Unit::TestCase
URI.parse(1) URI.parse(1)
end end
end end
def test_unescape
p1 = URI::Parser.new
assert_equal("\xe3\x83\x90", p1.unescape("\xe3\x83\x90"))
assert_equal("\xe3\x83\x90", p1.unescape('%e3%83%90'))
assert_equal("\xe3\x83\x90\xe3\x83\x90", p1.unescape("\xe3\x83\x90%e3%83%90"))
end
end end