Merge pull request #1184 from karuna/master

Hook url should be able to contain http basic authentication

1 files changed, 10 insertions, 1 deletions

diff --git a/app/models/web_hook.rb b/app/models/web_hook.rb
index 85d87898682..76efa50198b 100644
--- a/app/models/web_hook.rb
+++ b/app/models/web_hook.rb
@@ -11,7 +11,16 @@ class WebHook < ActiveRecord::Base
               message: "should be a valid url" }
 
   def execute(data)
-    WebHook.post(url, body: data.to_json, headers: { "Content-Type" => "application/json" })
+    parsed_url = URI.parse(url)
+    if parsed_url.userinfo.blank?
+      WebHook.post(url, body: data.to_json, headers: { "Content-Type" => "application/json" })
+    else
+      post_url = url.gsub(parsed_url.userinfo+"@", "")
+      WebHook.post(post_url,
+                   body: data.to_json,
+                   headers: { "Content-Type" => "application/json" }, 
+                   basic_auth: {username: parsed_url.user, password: parsed_url.password})
+    end
   end
   
 end