From dea106ccde8aee04fcbdf3fc79ae9159c505a603 Mon Sep 17 00:00:00 2001 From: Erwan Arzur Date: Fri, 29 Apr 2016 16:43:22 +0200 Subject: [PATCH] Fix Slack SSL connection (#83) Make HTTPClient auto-negotiate SSL supported ciphers - forget about :SSLV23 which are refused by most servers now. - log exceptions that could occur * remove comment --- lib/redmine_mattermost/listener.rb | 7 ++++--- 1 file changed, 4 insertions(+), 3 deletions(-) diff --git a/lib/redmine_mattermost/listener.rb b/lib/redmine_mattermost/listener.rb index 46fad38..c46510e 100644 --- a/lib/redmine_mattermost/listener.rb +++ b/lib/redmine_mattermost/listener.rb @@ -152,10 +152,11 @@ class MattermostListener < Redmine::Hook::Listener begin client = HTTPClient.new client.ssl_config.cert_store.set_default_paths - client.ssl_config.ssl_version = "SSLv23" + client.ssl_config.ssl_version = :auto client.post_async url, {:payload => params.to_json} - rescue - # Bury exception if connection error + rescue Exception => e + logger.warn("cannot connect to #{url}") + logger.warn(e) end end