[ruby-dev:34494] Re: OpenSSL::SSL::SSLContext#set_params

西山和広です。

At Wed, 23 Apr 2008 15:40:29 +0900,
Kazuhiro NISHIYAMA wrote:
> 
> IRCで聞いた
> svn diff -r 14269:14479 ext/openssl/lib/openssl/ssl.rb
> で試してみましたが、例外が変わっただけで動きませんでした。

さらにext/openssl/ossl_ssl.cをコピーするとpop3s対応は動くように
なったのですが、テストもbackportしようとしてtest/openssl/test_ssl.rbを
試すと1F1Eになりました。

  1) Error:
test_read_and_write(OpenSSL::TestSSL):
ArgumentError: wrong number of arguments (2 for 1)
    test/openssl/test_ssl.rb:204:in `gets'
    test/openssl/test_ssl.rb:204:in `test_read_and_write'
    test/openssl/test_ssl.rb:197:in `times'
    test/openssl/test_ssl.rb:197:in `test_read_and_write'
    test/openssl/test_ssl.rb:129:in `call'
    test/openssl/test_ssl.rb:129:in `start_server'
    test/openssl/test_ssl.rb:177:in `test_read_and_write'

  2) Failure:
test_sslctx_set_params(OpenSSL::TestSSL)
    [test/openssl/test_ssl.rb:353:in `test_sslctx_set_params'
     test/openssl/test_ssl.rb:129:in `call'
     test/openssl/test_ssl.rb:129:in `start_server'
     test/openssl/test_ssl.rb:343:in `test_sslctx_set_params']:
<false> is not true.

1Eはlimit付きのgetsが1.9 featuresなので、テストを削ればいいとして、
test_sslctx_set_paramsの方はset_paramsの実装がHashのキーの順番に
依存している(ssl_versionがciphersの前にあることを期待している)のに
        params.each{|name, value| self.__send__("#{name}=", value) }
でciphersの後にssl_versionが来てしまってFailureになっているようです。

順番に依存するものがssl_versionだけなら
        self.ssl_version = params.delete(:ssl_version)
を追加すれば良いかと思ったのですが、どうでしょうか?


-- 
|ZnZ(ゼット エヌ ゼット)
|西山和広(Kazuhiro NISHIYAMA)