Opened on 09/22/2014 at 10:41:57 AM

Closed on 09/22/2014 at 10:53:37 AM

#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).

Attachments (0)

Change History (2)

comment:1 Changed on 09/22/2014 at 10:50:36 AM by trev

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

comment:2 Changed on 09/22/2014 at 10:53:37 AM by trev

  • Resolution set to fixed
  • Status changed from reviewing to closed

Fixed: ​https://hg.adblockplus.org/sitescripts/rev/9f7c5a966fb0

Add Comment

Modify Ticket

Change Properties
Action
as closed .
The resolution will be deleted. Next status will be 'reopened'.
to The owner will be changed from trev.
Attachments ↑ Description ↑
 
Note: See TracTickets for help on using tickets.