TEST_CODEOWNERS/rails
3
0
Fork 0
Code Issues Pull Requests 1 Actions Packages Projects Releases Wiki Activity

Merge pull request #47081 from ioquatix/rack-3-query-parser-utf8

Browse Source 
Don't assume that invalid UTF-8 will raise an error.
This commit is contained in:
Rafael Mendonça França 2023-01-20 15:20:42 -05:00 committed by GitHub
commit 72642bd74b
No known key found for this signature in database
GPG Key ID: 4AEE18F83AFDEB23
1 changed files with 13 additions and 11 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

2
actionpack/test/dispatch/request_test.rb
View File

@ -1131,6 +1131,7 @@ class RequestParameters < BaseRequestTest
end
end
if Rack.release < "3"
test "parameters not accessible after rack parse error of invalid UTF8 character" do
request = stub_request("QUERY_STRING" => "foo%81E=1")
assert_raises(ActionController::BadRequest) { request.parameters }
@ -1145,6 +1146,7 @@ class RequestParameters < BaseRequestTest
request = stub_request("QUERY_STRING" => "foo[bar]=%81E")
assert_raises(ActionController::BadRequest) { request.parameters }
end
end
test "POST parameters containing invalid UTF8 character" do
data = "foo=%81E"