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

http-client-tls test failure #196

Closed
bergmark opened this issue May 7, 2016 · 7 comments
Closed

http-client-tls test failure #196

bergmark opened this issue May 7, 2016 · 7 comments

Comments

@bergmark
Copy link
Contributor

bergmark commented May 7, 2016

Perhaps because of the new version of tls?

Building http-client-tls-0.2.4...
Preprocessing library http-client-tls-0.2.4...
[1 of 1] Compiling Network.HTTP.Client.TLS ( Network/HTTP/Client/TLS.hs, dist/build/Network/HTTP/Client/TLS.o )
In-place registering http-client-tls-0.2.4...
Preprocessing test suite 'spec' for http-client-tls-0.2.4...
[1 of 1] Compiling Main             ( test/Spec.hs, dist/build/spec/spec-tmp/Main.o )
Linking dist/build/spec/spec ...
> /tmp/stackage-build8/http-client-tls-0.2.4$ dist/build/spec/spec

make a TLS connection FAILED [1]

Failures:

  test/Spec.hs:10:
  1) make a TLS connection
       uncaught exception: HttpException (TlsExceptionHostPort (HandshakeFailed (Error_Packet_unexpected "Alert [(AlertLevel_Fatal,BadRecordMac)]" " expected: change cipher")) "httpbin.org" 443)
@snoyberg
Copy link
Owner

snoyberg commented May 7, 2016

Pinging @vincenthz

@creichert
Copy link
Contributor

creichert commented May 7, 2016

This issue was fixed here: haskell-tls/hs-tls#124

I would consider this somewhat serious as I had multiple clients & servers deployed with this issue from a stackage snapshot which were practically unusable under load. Now I'm using:

extra-deps:
  # Pin tls version to fix BadRecordMac exceptions in http-client-tls
  # https://github.com/vincenthz/hs-tls/issues/124
  - tls-1.3.5
  - cryptonite-0.15

@snoyberg
Copy link
Owner

snoyberg commented May 7, 2016

This shouldn't be showing up in recent builds though, since Stackage has already moved over to a recent tls release.

@creichert
Copy link
Contributor

What stackage snapshot is being used when the above fails? http-client-tls-0.2.0.4 has been in several versions of stackage yet the tls package was upgraded to a "fixed" vesion in lts-5.14.

@bergmark
Copy link
Contributor Author

bergmark commented May 7, 2016

this is while building a new nightly, with tls-1.3.6

@vincenthz
Copy link
Contributor

vincenthz commented May 8, 2016

looks like a bug in SHA384 based ciphers which got defined while investigating haskell-tls/hs-tls#139 which made tls 1.3.6; I've disabled them now, and uploaded tls 1.3.7

bergmark added a commit to commercialhaskell/stackage that referenced this issue May 8, 2016
@bergmark
Copy link
Contributor Author

LTS 5.16 made it out with http-client-tls-0.2.4 + tls-1.3.7 so this is fixed, thanks!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

4 participants