Merge pull request #5760 from dermoth/master

Fix HTTP push to public repos

Merge pull request #5760 from dermoth/master
Fix HTTP push to public repos
1ad2a936 · Dmitriy Zaporozhets · a00874a9 · 843548cc · 1ad2a936
Commit 1ad2a936 authored Jan 07, 2014 by Dmitriy Zaporozhets
Show whitespace changes
Inline Side-by-side

Showing with 3 additions and 0 deletions

lib/gitlab/backend/grack_auth.rb lib/gitlab/backend/grack_auth.rb +3 -0

No files found.
--- a/lib/gitlab/backend/grack_auth.rb
+++ b/lib/gitlab/backend/grack_auth.rb
@@ -92,6 +92,9 @@ module Grack
          return false unless can?(user, action, project)
        end

+        # Never let git-receive-pack trough unauthenticated; it's
+        # harmless but git < 1.8 doesn't like it
+        return false if user.nil?
        true
      else
        false