From aeb2f49fd4c65b9f5e3b2d7858a250a747a0f702 Mon Sep 17 00:00:00 2001 From: digitalMoksha Date: Tue, 21 Nov 2017 15:35:30 +0100 Subject: [PATCH] Revert "check for `read_only?` first before seeing if request is disallowed" This reverts commit 91075c8237307e09c2be8a88ffb3711fd62417d1. --- lib/gitlab/middleware/read_only.rb | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/lib/gitlab/middleware/read_only.rb b/lib/gitlab/middleware/read_only.rb index f8b9c5bff4c..c26656704d7 100644 --- a/lib/gitlab/middleware/read_only.rb +++ b/lib/gitlab/middleware/read_only.rb @@ -14,7 +14,7 @@ module Gitlab @env = env @route_hash = nil - if Gitlab::Database.read_only? && disallowed_request? + if disallowed_request? && Gitlab::Database.read_only? Rails.logger.debug('GitLab ReadOnly: preventing possible non read-only operation') error_message = 'You cannot do writing operations on a read-only GitLab instance'