Opened 5 years ago

Closed 5 years ago

#1426 closed change (fixed)

Don't ignore redirects during stats processing

Reported by: trev Assignee: trev
Priority: P2 Milestone:
Module: Infrastructure Keywords:
Cc: Blocked By:
Blocking: Platform: Unknown
Ready: yes Confidential: no
Tester: Verified working: no
Review URL(s):

http://codereview.adblockplus.org/6312256635666432

Description

Background

In order to reduce the noise in the stats we ignore any response codes other than 200. This means however that we cannot see redirect locations in our stats.

What to change

Accept response codes 301 and 302 as well (mostly the latter actually).

Change History (2)

comment:1 Changed 5 years ago by trev

  • Review URL(s) modified (diff)
  • Status changed from new to reviewing

comment:2 Changed 5 years ago by trev

  • Resolution set to fixed
  • Status changed from reviewing to closed
Note: See TracTickets for help on using tickets.