Download as
Browse Code »

Commit 67f6ebd913c3b64ae82f47ce6835ab54f8291eaf

Authored by Rodrigo Souto
1 parent 4e97f113
Exists in master and in 22 other branches angular_poc, api-article-archived, article-readonly, backup-7-jan-2016, captcha_serpro_plugin, content-manager-hostspot, export-comment-paragraph, fix_event_date_issue, login-captcha, master_rails3, new_video_plugin, pg_search_rank, production, refactor_with_role, refactor_with_role_scope, remove_profile_cat_icons, resend_confirmation_email, staging, staging_rails3, tasks_keep_filter_params, theme-brasil-digital-from-staging, travis

content-viewer-controller: user_is_a_bot consider possible empty user_agent

Showing 1 changed file with 6 additions and 4 deletions   Show diff stats
app/controllers/public/content_viewer_controller.rb
  Diff comments   View file @67f6ebd
... ... @@ -159,10 +159,12 @@ class ContentViewerController < ApplicationController
159 159 helper_method :pass_without_comment_captcha?
160 160  
161 161 def user_is_a_bot?
162   - request.env["HTTP_USER_AGENT"].match(/bot/) ||
163   - request.env["HTTP_USER_AGENT"].match(/spider/) ||
164   - request.env["HTTP_USER_AGENT"].match(/crawler/) ||
165   - request.env["HTTP_USER_AGENT"].match(/\(.*https?:\/\/.*\)/)
  162 + user_agent= request.env["HTTP_USER_AGENT"]
  163 + user_agent.blank? ||
  164 + user_agent.match(/bot/) ||
  165 + user_agent.match(/spider/) ||
  166 + user_agent.match(/crawler/) ||
  167 + user_agent.match(/\(.*https?:\/\/.*\)/)
166 168 end
167 169  
168 170 end
... ...