Skip to content

Commit f89ce51

Browse files
committed
Update error message for request_specific_fields option validation
Added `inspect` to the `request_specific_fields` value to provide better visibility for users in the exception message.
1 parent 2e7734c commit f89ce51

File tree

2 files changed

+2
-2
lines changed

2 files changed

+2
-2
lines changed

lib/open-uri.rb

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -151,7 +151,7 @@ def OpenURI.open_uri(name, *rest) # :nodoc:
151151
end
152152
if options.has_key? :request_specific_fields
153153
if !(options[:request_specific_fields].is_a?(Hash) || options[:request_specific_fields].is_a?(Proc))
154-
raise ArgumentError, "Invalid request_specific_fields' format: #{options[:request_specific_fields]}"
154+
raise ArgumentError, "Invalid request_specific_fields option: #{options[:request_specific_fields].inspect}"
155155
end
156156
end
157157
unless mode == nil ||

test/open-uri/test_open-uri.rb

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -393,7 +393,7 @@ def test_redirect_with_invalid_request_specific_fields_format
393393
with_http {|srv, url|
394394
srv.mount_proc("/r1/", lambda {|req, res| res.body = "r1" } )
395395
exc = assert_raise(ArgumentError) { URI.open("#{url}/r1/", request_specific_fields: "dummy_token") {} }
396-
assert_equal("Invalid request_specific_fields' format: dummy_token", exc.message)
396+
assert_equal('Invalid request_specific_fields option: "dummy_token"', exc.message)
397397
}
398398
end
399399

0 commit comments

Comments
 (0)