diff options
author | Jacob Vosmaer <contact@jacobvosmaer.nl> | 2016-05-10 11:14:46 +0200 |
---|---|---|
committer | Jacob Vosmaer <contact@jacobvosmaer.nl> | 2016-05-10 11:14:46 +0200 |
commit | ebf80db3abcaf4a0e08273bf180aa33368610b8a (patch) | |
tree | 4c5ea567ba9d1498d345f3ce53d0f04f0b6b37d2 /app/models/hooks | |
parent | a19508d6b79aa3a399f479b09b52b35b5289a92a (diff) | |
download | gitlab-ce-ebf80db3abcaf4a0e08273bf180aa33368610b8a.tar.gz |
Remove extra sanitization
Robert Speicher and I believe this string gets sanitized further down
the stack anyway. Doing this in a model class feels wrong.
Diffstat (limited to 'app/models/hooks')
-rw-r--r-- | app/models/hooks/web_hook.rb | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/app/models/hooks/web_hook.rb b/app/models/hooks/web_hook.rb index 818abbf4cc4..a5ac8bee6b9 100644 --- a/app/models/hooks/web_hook.rb +++ b/app/models/hooks/web_hook.rb @@ -59,7 +59,7 @@ class WebHook < ActiveRecord::Base basic_auth: auth) end - [response.code, ActionView::Base.full_sanitizer.sanitize(response.to_s)] + [response.code, response.to_s] rescue SocketError, OpenSSL::SSL::SSLError, Errno::ECONNRESET, Errno::ECONNREFUSED, Net::OpenTimeout => e logger.error("WebHook Error => #{e}") [false, e.to_s] |