diff options
author | GitLab Bot <gitlab-bot@gitlab.com> | 2021-07-20 12:55:51 +0300 |
---|---|---|
committer | GitLab Bot <gitlab-bot@gitlab.com> | 2021-07-20 12:55:51 +0300 |
commit | e8d2c2579383897a1dd7f9debd359abe8ae8373d (patch) | |
tree | c42be41678c2586d49a75cabce89322082698334 /app/services/user_agent_detail_service.rb | |
parent | fc845b37ec3a90aaa719975f607740c22ba6a113 (diff) |
Add latest changes from gitlab-org/gitlab@14-1-stable-eev14.1.0-rc42
Diffstat (limited to 'app/services/user_agent_detail_service.rb')
-rw-r--r-- | app/services/user_agent_detail_service.rb | 17 |
1 files changed, 11 insertions, 6 deletions
diff --git a/app/services/user_agent_detail_service.rb b/app/services/user_agent_detail_service.rb index 9302c86d3e6..01a98a15869 100644 --- a/app/services/user_agent_detail_service.rb +++ b/app/services/user_agent_detail_service.rb @@ -1,16 +1,21 @@ # frozen_string_literal: true class UserAgentDetailService - attr_accessor :spammable, :request - - def initialize(spammable, request) + def initialize(spammable:, spam_params:) @spammable = spammable - @request = request + @spam_params = spam_params end def create - return unless request + unless spam_params&.user_agent && spam_params&.ip_address + messasge = 'Skipped UserAgentDetail creation because necessary spam_params were not provided' + return ServiceResponse.success(message: messasge) + end - spammable.create_user_agent_detail(user_agent: request.env['HTTP_USER_AGENT'], ip_address: request.env['action_dispatch.remote_ip'].to_s) + spammable.create_user_agent_detail(user_agent: spam_params.user_agent, ip_address: spam_params.ip_address) end + + private + + attr_reader :spammable, :spam_params end |