diff options
author | Mike Greiling <mike@pixelcog.com> | 2017-10-24 10:47:03 +0300 |
---|---|---|
committer | Mike Greiling <mike@pixelcog.com> | 2017-10-24 10:47:03 +0300 |
commit | 6d0045d2671c11854ee1e34609f28526470f5ee2 (patch) | |
tree | 30d7783f20d1d4d643b369dbf3d6e25e5379d0b3 | |
parent | 0258a37f07f9c8e788bf37e14833d7accf16ed9d (diff) | |
download | gitlab-ce-6d0045d2671c11854ee1e34609f28526470f5ee2.tar.gz |
fix rubocop violations
-rw-r--r-- | lib/gitlab/testing/request_inspector_middleware.rb | 15 |
1 files changed, 9 insertions, 6 deletions
diff --git a/lib/gitlab/testing/request_inspector_middleware.rb b/lib/gitlab/testing/request_inspector_middleware.rb index ff5bf9d74f4..a5ac800c735 100644 --- a/lib/gitlab/testing/request_inspector_middleware.rb +++ b/lib/gitlab/testing/request_inspector_middleware.rb @@ -1,3 +1,5 @@ +# rubocop:disable Style/ClassVars + module Gitlab module Testing class RequestInspectorMiddleware @@ -30,12 +32,13 @@ module Gitlab request_headers = env_http_headers(env) status, headers, body = @app.call(env) - log_response(OpenStruct.new( + request = OpenStruct.new( url: url, status_code: status, request_headers: request_headers, response_headers: headers - )) + ) + log_request request [status, headers, body] end @@ -43,14 +46,14 @@ module Gitlab private def env_http_headers(env) - Hash[*env.select {|k,v| k.start_with? 'HTTP_'} - .collect {|k,v| [k.sub(/^HTTP_/, ''), v]} - .collect {|k,v| [k.split('_').collect(&:capitalize).join('-'), v]} + Hash[*env.select {|k, v| k.start_with? 'HTTP_'} + .collect {|k, v| [k.sub(/^HTTP_/, ''), v]} + .collect {|k, v| [k.split('_').collect(&:capitalize).join('-'), v]} .sort .flatten] end - def log_response(response) + def log_request(response) @@logged_requests.push(response) end end |