ruby-changes:2166
From: ko1@a...
Date: 8 Oct 2007 20:17:10 +0900
Subject: [ruby-changes:2166] gotoyuzo - Ruby:r13657 (ruby_1_8): * lib/net/http.rb, lib/open-uri.rb: remove
gotoyuzo 2007-10-08 20:16:54 +0900 (Mon, 08 Oct 2007)
New Revision: 13657
Modified files:
branches/ruby_1_8/ChangeLog
branches/ruby_1_8/ext/openssl/lib/net/ftptls.rb
branches/ruby_1_8/ext/openssl/lib/net/telnets.rb
branches/ruby_1_8/lib/net/http.rb
branches/ruby_1_8/lib/net/imap.rb
branches/ruby_1_8/lib/open-uri.rb
Log:
* lib/net/http.rb, lib/open-uri.rb: remove
Net::HTTP#enable_post_connection_check. [ruby-dev:31960]
* lib/net/imap.rb: hostname should be verified against server's
indentity as persented in the server's certificate. [ruby-dev:31960]
* ext/openssl/lib/net/telnets.rb, ext/openssl/lib/net/ftptls.rb: ditto.
http://svn.ruby-lang.org/cgi-bin/viewvc.cgi/branches/ruby_1_8/ChangeLog?r1=13657&r2=13656
http://svn.ruby-lang.org/cgi-bin/viewvc.cgi/branches/ruby_1_8/ext/openssl/lib/net/ftptls.rb?r1=13657&r2=13656
http://svn.ruby-lang.org/cgi-bin/viewvc.cgi/branches/ruby_1_8/lib/open-uri.rb?r1=13657&r2=13656
http://svn.ruby-lang.org/cgi-bin/viewvc.cgi/branches/ruby_1_8/lib/net/imap.rb?r1=13657&r2=13656
http://svn.ruby-lang.org/cgi-bin/viewvc.cgi/branches/ruby_1_8/lib/net/http.rb?r1=13657&r2=13656
http://svn.ruby-lang.org/cgi-bin/viewvc.cgi/branches/ruby_1_8/ext/openssl/lib/net/telnets.rb?r1=13657&r2=13656
Index: ruby_1_8/ext/openssl/lib/net/telnets.rb
===================================================================
--- ruby_1_8/ext/openssl/lib/net/telnets.rb (revision 13656)
+++ ruby_1_8/ext/openssl/lib/net/telnets.rb (revision 13657)
@@ -134,6 +134,9 @@
@sock.verify_callback = @options['VerifyCallback']
@sock.verify_depth = @options['VerifyDepth']
@sock.connect
+ if @options['VerifyMode'] != OpenSSL::SSL::VERIFY_NONE
+ @sock.post_connection_check(@options['Host'])
+ end
@ssl = true
end
''
Index: ruby_1_8/ext/openssl/lib/net/ftptls.rb
===================================================================
--- ruby_1_8/ext/openssl/lib/net/ftptls.rb (revision 13656)
+++ ruby_1_8/ext/openssl/lib/net/ftptls.rb (revision 13657)
@@ -29,13 +29,23 @@
module Net
class FTPTLS < FTP
+ def connect(host, port=FTP_PORT)
+ @hostname = host
+ super
+ end
+
def login(user = "anonymous", passwd = nil, acct = nil)
+ store = OpenSSL::X509::Store.new
+ store.set_default_paths
ctx = OpenSSL::SSL::SSLContext.new('SSLv23')
+ ctx.cert_store = store
+ ctx.verify_mode = OpenSSL::SSL::VERIFY_PEER
ctx.key = nil
ctx.cert = nil
voidcmd("AUTH TLS")
@sock = OpenSSL::SSL::SSLSocket.new(@sock, ctx)
@sock.connect
+ @sock.post_connection_check(@hostname)
super(user, passwd, acct)
voidcmd("PBSZ 0")
end
Index: ruby_1_8/ChangeLog
===================================================================
--- ruby_1_8/ChangeLog (revision 13656)
+++ ruby_1_8/ChangeLog (revision 13657)
@@ -1,3 +1,13 @@
+Mon Oct 8 20:06:23 2007 GOTOU Yuuzou <gotoyuzo@n...>
+
+ * lib/net/http.rb, lib/open-uri.rb: remove
+ Net::HTTP#enable_post_connection_check. [ruby-dev:31960]
+
+ * lib/net/imap.rb: hostname should be verified against server's
+ indentity as persented in the server's certificate. [ruby-dev:31960]
+
+ * ext/openssl/lib/net/telnets.rb, ext/openssl/lib/net/ftptls.rb: ditto.
+
Sat Oct 6 23:14:54 2007 Yukihiro Matsumoto <matz@r...>
* string.c (rb_str_to_i): update RDoc since base can be any value
Index: ruby_1_8/lib/open-uri.rb
===================================================================
--- ruby_1_8/lib/open-uri.rb (revision 13656)
+++ ruby_1_8/lib/open-uri.rb (revision 13657)
@@ -229,7 +229,6 @@
if target.class == URI::HTTPS
require 'net/https'
http.use_ssl = true
- http.enable_post_connection_check = true
http.verify_mode = OpenSSL::SSL::VERIFY_PEER
store = OpenSSL::X509::Store.new
store.set_default_paths
Index: ruby_1_8/lib/net/http.rb
===================================================================
--- ruby_1_8/lib/net/http.rb (revision 13656)
+++ ruby_1_8/lib/net/http.rb (revision 13657)
@@ -470,7 +470,6 @@
@debug_output = nil
@use_ssl = false
@ssl_context = nil
- @enable_post_connection_check = true
end
def inspect
@@ -527,9 +526,6 @@
false # redefined in net/https
end
- # specify enabling SSL server certificate and hostname checking.
- attr_accessor :enable_post_connection_check
-
# Opens TCP connection and HTTP session.
#
# When this method is called with block, gives a HTTP object
@@ -589,12 +585,7 @@
end
s.connect
if @ssl_context.verify_mode != OpenSSL::SSL::VERIFY_NONE
- begin
- s.post_connection_check(@address)
- rescue OpenSSL::SSL::SSLError => ex
- raise ex if @enable_post_connection_check
- warn ex.message
- end
+ s.post_connection_check(@address)
end
end
on_connect
Index: ruby_1_8/lib/net/imap.rb
===================================================================
--- ruby_1_8/lib/net/imap.rb (revision 13656)
+++ ruby_1_8/lib/net/imap.rb (revision 13657)
@@ -900,6 +900,7 @@
end
@sock = SSLSocket.new(@sock, context)
@sock.connect # start ssl session.
+ @sock.post_connection_check(@host) if verify
else
@usessl = false
end
--
ML: ruby-changes@q...
Info: http://www.atdot.net/~ko1/quickml