Skip to content

Commit

Permalink
[ruby/webrick] Skip env-locale-sensitive CGI test on the "java" platform
Browse files Browse the repository at this point in the history
JRuby's environment variables are provided by the Java Development
Kit's (JDK's) classes, which present them as a map from string to
string. In order to do this, those environment variable names and
values must be decoded into characters, which breaks any variables
that are intended to be "raw" bytes not necessarily decodable with
the default system encoding.

This issue is detailed in jruby/jruby#6248. The only solution on
the JRuby side will be to bypass the JDK environment variable API
and go directly to the native getenv/setenv system calls. This is
not likely to happen in the near future, due to the complexity of
such a change and the rarity of undecodable environment values.

The exclude here was added due to the Windows platform also having
a similar sensitivity to character encodings when working with
environment variables. It seems appropriate to expand this skip
to the "java" platform, as the root issue is largely the same.

ruby/webrick@dc453e5c3c
  • Loading branch information
headius authored and hsbt committed Sep 24, 2020
1 parent 0c611d7 commit 46ba416
Showing 1 changed file with 1 addition and 1 deletion.
2 changes: 1 addition & 1 deletion test/webrick/test_cgi.rb
Expand Up @@ -43,7 +43,7 @@ def test_cgi
http.request(req){|res| assert_equal("/path/info", res.body, log.call)}
req = Net::HTTP::Get.new("/webrick.cgi/%3F%3F%3F?foo=bar")
http.request(req){|res| assert_equal("/???", res.body, log.call)}
unless RUBY_PLATFORM =~ /mswin|mingw|cygwin|bccwin32/
unless RUBY_PLATFORM =~ /mswin|mingw|cygwin|bccwin32|java/
# Path info of res.body is passed via ENV.
# ENV[] returns different value on Windows depending on locale.
req = Net::HTTP::Get.new("/webrick.cgi/%A4%DB%A4%B2/%A4%DB%A4%B2")
Expand Down

0 comments on commit 46ba416

Please sign in to comment.