diff options
author | Anatolii Didukh <railsme7@gmail.com> | 2016-06-06 19:50:05 +0300 |
---|---|---|
committer | Anatolii Didukh <railsme7@gmail.com> | 2016-06-06 22:47:52 +0300 |
commit | ba67b778d54b7213c0e7f1a9073885fe5117aeb2 (patch) | |
tree | 23b545b1652bfdb3261dda6e94d06be61285516c | |
parent | 76e2d74769df6258fced3ed9761c2f0e849483f0 (diff) | |
download | rack-ba67b778d54b7213c0e7f1a9073885fe5117aeb2.tar.gz |
replace first and last characters check
-rw-r--r-- | lib/rack/media_type.rb | 2 | ||||
-rw-r--r-- | lib/rack/multipart/parser.rb | 2 | ||||
-rw-r--r-- | lib/rack/request.rb | 2 |
3 files changed, 3 insertions, 3 deletions
diff --git a/lib/rack/media_type.rb b/lib/rack/media_type.rb index 7e6cd3a8..f9397453 100644 --- a/lib/rack/media_type.rb +++ b/lib/rack/media_type.rb @@ -31,7 +31,7 @@ module Rack private def strip_doublequotes(str) - (str[0] == ?" && str[-1] == ?") ? str[1..-2] : str + (str.start_with?('"') && str.end_with?('"')) ? str[1..-2] : str end end end diff --git a/lib/rack/multipart/parser.rb b/lib/rack/multipart/parser.rb index 74a7ee67..f661da10 100644 --- a/lib/rack/multipart/parser.rb +++ b/lib/rack/multipart/parser.rb @@ -345,7 +345,7 @@ module Rack k,v = param.split('=', 2) k.strip! v.strip! - v = v[1..-2] if v[0] == '"' && v[-1] == '"' + v = v[1..-2] if v.start_with?('"') && v.end_with?('"') encoding = Encoding.find v if k == CHARSET end end diff --git a/lib/rack/request.rb b/lib/rack/request.rb index 5bf3eb17..e2d2bb56 100644 --- a/lib/rack/request.rb +++ b/lib/rack/request.rb @@ -337,7 +337,7 @@ module Rack # Fix for Safari Ajax postings that always append \0 # form_vars.sub!(/\0\z/, '') # performance replacement: - form_vars.slice!(-1) if form_vars[-1] == ?\0 + form_vars.slice!(-1) if form_vars.end_with?("\0") set_header RACK_REQUEST_FORM_VARS, form_vars set_header RACK_REQUEST_FORM_HASH, parse_query(form_vars, '&') |