Bug #5950

Feature #859: open-uri doesn't allow redirection to https

open-uri: https redirect fix

Added by Aaron Zauner about 2 years ago. Updated about 1 year ago.

[ruby-core:42268]
Status:Assigned
Priority:Low
Assignee:Akira Tanaka
Category:lib
Target version:-
ruby -v:ruby 1.8.7 (2011-06-30 patchlevel 352) [x86_64-linux] Backport:

Description

open-uri raises an exception if a http/s redirect refers to https.

original mail to the maintainer with a quickfix: https://gist.github.com/1704932

--snip--

this is taken from the original ruby open-uri class,

fixed this to support secure socket http redirects:

def OpenURI.redirectable?(uri1, uri2) # :nodoc:
# This test is intended to forbid a redirection from http://... to
# file:///etc/passwd.
# However this is ad hoc. It should be extensible/configurable.
uri1.scheme.downcase == uri2.scheme.downcase ||
(/\A(?:http|ftp|https)\z/i =~ uri1.scheme && /\A(?:http|ftp|https)\z/i =~
uri2.scheme)
end
--snip--

History

#1 Updated by Jakob Hilden almost 2 years ago

Seems like a duplicate of http://bugs.ruby-lang.org/issues/859

#2 Updated by Yusuke Endoh almost 2 years ago

  • Status changed from Open to Assigned
  • Assignee set to Akira Tanaka

#3 Updated by Akira Tanaka about 1 year ago

  • Project changed from Ruby 1.8 to ruby-trunk
  • Category changed from lib to lib
  • Target version deleted (Ruby 1.8.7)

#4 Updated by Akira Tanaka about 1 year ago

  • Parent task set to #859

Also available in: Atom PDF