Skip to content

Commit

Permalink
Remove honor_cipher_order option
Browse files Browse the repository at this point in the history
A new hackney version has been released which no longer sets
`honor_cipher_order` to `true` by default. This means we can remove our
workaround. Make sure to upgrade hackney for AppSignal to work.
  • Loading branch information
tombruijn committed Sep 23, 2019
1 parent 35535d3 commit b204685
Show file tree
Hide file tree
Showing 2 changed files with 2 additions and 4 deletions.
3 changes: 1 addition & 2 deletions lib/appsignal/transmitter.ex
Original file line number Diff line number Diff line change
Expand Up @@ -18,8 +18,7 @@ defmodule Appsignal.Transmitter do
[
ssl_options: [
cacertfile: ca_file_path,
ciphers: ciphers(),
honor_cipher_order: :undefined
ciphers: ciphers()
]
]}

Expand Down
3 changes: 1 addition & 2 deletions mix_helpers.exs
Original file line number Diff line number Diff line change
Expand Up @@ -190,8 +190,7 @@ defmodule Mix.Appsignal.Helper do
options = [
ssl_options: [
cacertfile: priv_path("cacert.pem"),
ciphers: ciphers(),
honor_cipher_order: :undefined
ciphers: ciphers()
]
]

Expand Down

0 comments on commit b204685

Please sign in to comment.