Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

FAIL: test_wantWriteError on FreeBSD (pyopenssl 0.14) #37

Closed
koobs opened this issue Feb 24, 2014 · 7 comments · Fixed by #76
Closed

FAIL: test_wantWriteError on FreeBSD (pyopenssl 0.14) #37

koobs opened this issue Feb 24, 2014 · 7 comments · Fixed by #76

Comments

@koobs
Copy link

koobs commented Feb 24, 2014

I'm not sure that #31 is actually fixed, but opted to create a new issue instead:

OS: FreeBSD 9.2-STABLE #0 r260870
Python: 2.7.6

======================================================================
FAIL: test_wantWriteError (OpenSSL.test.test_ssl.ConnectionTests)
----------------------------------------------------------------------
Traceback (most recent call last):
  File "/mnt/home/user/repos/freebsd/ports/security/py-openssl/work/pyca-pyopenssl-0146d44/OpenSSL/test/test_ssl.py", line 1931, in test_wantWriteError
    self.assertRaises(WantWriteError, conn.do_handshake)
  File "/mnt/home/user/repos/freebsd/ports/security/py-openssl/work/pyca-pyopenssl-0146d44/OpenSSL/test/util.py", line 266, in failUnlessRaises
    exception.__name__,
AssertionError: <class 'OpenSSL.SSL.WantReadError'> raised instead of WantWriteError
----------------------------------------------------------------------
@kouk
Copy link
Contributor

kouk commented Feb 24, 2014

@koobs what does sysctl net.inet.tcp.sendspace net.inet.tcp.recvspace print on your system?

@koobs
Copy link
Author

koobs commented Feb 24, 2014

net.inet.tcp.sendspace: 65536
net.inet.tcp.recvspace: 32768

@kouk
Copy link
Contributor

kouk commented Feb 25, 2014

huh, weird. compared to yours my values are interchanged.

kouk added a commit to kouk/pyopenssl that referenced this issue Feb 25, 2014
@koobs
Copy link
Author

koobs commented Feb 25, 2014

@kouk I was going to mention it in my earlier reply, these (my) values are non-default, increasing send buffers in favour of smaller receive buffers optimized for web serving (small requests, large responses)

@exarkun
Copy link
Member

exarkun commented Mar 22, 2014

Can you give the branch for #76 a try?

@koobs
Copy link
Author

koobs commented Mar 22, 2014

test_wantWriteError (OpenSSL.test.test_ssl.ConnectionTests) ... ok
...
Ran 353 tests in 1.944s
FAILED (failures=9, errors=1)

Looks good :)

Failures (unrelated, but for point-in-time reference):

FAIL: test_set_verify_callback_exception (OpenSSL.test.test_ssl.ContextTests)
FAIL: test_digest (OpenSSL.test.test_crypto.X509Tests)
FAIL: test_various_empty_passphrases (OpenSSL.test.test_crypto.PKCS12Tests)
FAIL: test_friendly_name (OpenSSL.test.test_crypto.PKCS12Tests)
FAIL: test_export_without_args (OpenSSL.test.test_crypto.PKCS12Tests)
FAIL: test_dump_certificate_request (OpenSSL.test.test_crypto.FunctionTests)
FAIL: test_dump_certificate (OpenSSL.test.test_crypto.FunctionTests)
FAIL: test_export (OpenSSL.test.test_crypto.CRLTests)
ERROR: test_set_default_verify_paths (OpenSSL.test.test_ssl.ContextTests)

@kouk
Copy link
Contributor

kouk commented Mar 24, 2014

#76 works for me as well.

@github-actions github-actions bot locked as resolved and limited conversation to collaborators Aug 20, 2020
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
3 participants