Commit 6957fb7c authored by Jacob Vosmaer's avatar Jacob Vosmaer

Always mention HTTP status

parent 23a3e375
...@@ -28,7 +28,7 @@ class Projects::HooksController < Projects::ApplicationController ...@@ -28,7 +28,7 @@ class Projects::HooksController < Projects::ApplicationController
status, message = TestHookService.new.execute(hook, current_user) status, message = TestHookService.new.execute(hook, current_user)
if status && status >= 200 && status < 400 if status && status >= 200 && status < 400
flash[:notice] = "Hook executed successfully" flash[:notice] = "Hook executed successfully: HTTP #{status}"
elsif status elsif status
flash[:alert] = "Hook executed successfully but returned HTTP #{status} #{message}" flash[:alert] = "Hook executed successfully but returned HTTP #{status} #{message}"
else else
......
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment