Clarify protocol access check, and make Git HTTP access call more specific.

app/controllers/projects/git_http_controller.rb

@@ -157,7 +157,7 @@ class Projects::GitHttpController < Projects::ApplicationController
   end
   def render_not_allowed
-    render plain: access.message, status: :forbidden
+    render plain: download_access.message, status: :forbidden
   end
   def ci?
@@ -168,20 +168,20 @@ class Projects::GitHttpController < Projects::ApplicationController
     return false unless Gitlab.config.gitlab_shell.upload_pack
     if user
-      access.allowed?
+      download_access.allowed?
     else
       ci? || project.public?
     end
   end
-  def access
-    return @access if defined?(@access)
-    @access = Gitlab::GitAccess.new(user, project, 'http').check('git-upload-pack')
+  def download_access
+    return @download_access if defined?(@download_access)
+    @download_access = Gitlab::GitAccess.new(user, project, 'http').check('git-upload-pack')
   end
   def http_blocked?
-    access.message.include?('HTTP')
+    download_access.protocol_allowed?
   end
   def receive_pack_allowed?

lib/gitlab/protocol_access.rb

@@ -3,7 +3,7 @@ module Gitlab
     def self.allowed?(protocol)
       if protocol.to_s == 'web'
         true
-      elsif !current_application_settings.enabled_git_access_protocols.present?
+      elsif current_application_settings.enabled_git_access_protocols.blank?
         true
       else
         protocol.to_s == current_application_settings.enabled_git_access_protocols