Merge branch 'custom-hook-output' into 'master'

Prevent blocking custom hook output streams

Based on https://gitlab.com/gitlab-org/gitlab-shell/merge_requests/49

Fixes https://gitlab.com/gitlab-org/gitlab-ce/issues/1458

See merge request !67

lib/gitlab_custom_hook.rb

@@ -25,37 +25,24 @@ class GitlabCustomHook
   private
 
   def call_receive_hook(hook, changes)
-    # function  will return true if succesful
-    exit_status = false
+    # Prepare the hook subprocess. Attach a pipe to its stdin, and merge
+    # both its stdout and stderr into our own stdout.
+    stdin_reader, stdin_writer = IO.pipe
+    hook_pid = spawn(hook, in: stdin_reader, err: :out)
+    stdin_reader.close
 
-    # we combine both stdout and stderr as we don't know what stream
-    # will be used by the custom hook
-    Open3.popen2e(hook) do |stdin, stdout_stderr, wait_thr|
-      exit_status = true
-      stdin.sync = true
-
-      # in git, pre- and post- receive hooks may just exit without
-      # reading stdin. We catch the exception to avoid a broken pipe
-      # warning
+    # Submit changes to the hook via its stdin.
     begin
-        # inject all the changes as stdin to the hook
-        changes.lines do |line|
-          stdin.puts(line)
-        end
+      IO.copy_stream(StringIO.new(changes), stdin_writer)
     rescue Errno::EPIPE
+      # It is not an error if the hook does not consume all of its input.
     end
 
-      # need to close stdin before reading stdout
-      stdin.close
-
-      unless wait_thr.value == 0
-        exit_status = false
-      end
-
-      stdout_stderr.each_line { |line| puts line }
-    end
+    # Close the pipe to let the hook know there is no further input.
+    stdin_writer.close
 
-    exit_status
+    Process.wait(hook_pid)
+    $?.success?
   end
 
   def hook_file(hook_type, repo_path)