Opened on 10/17/2016 at 11:15:14 AM

Closed on 10/17/2016 at 11:30:27 AM

Last modified on 10/31/2016 at 12:00:53 PM

#4538 closed change (fixed)

When exporting filters, mark $websocket filters as requiring Adblock Plus 2.8

Reported by: trev Assignee: trev
Priority: P1 Milestone: Adblock-Plus-2.8-for-Firefox
Module: Adblock-Plus-for-Firefox Keywords:
Cc: Ross Blocked By:
Blocking: Platform: Unknown / Cross platform
Ready: yes Confidential: no
Tester: Unknown Verified working: no
Review URL(s):

Description (last modified by trev)


We introduced $websocket filter type in #4449, meaning that it will be released with Adblock Plus 2.8.

What to change

When exporting custom filters only, make sure to use [Adblock Plus 2.8] header whenever a $websocket filter is present.

Attachments (0)

Change History (6)

comment:1 Changed on 10/17/2016 at 11:19:04 AM by trev

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

comment:2 Changed on 10/17/2016 at 11:29:41 AM by abpbot

comment:3 Changed on 10/17/2016 at 11:30:27 AM by trev

  • Milestone set to Adblock-Plus-2.8-for-Firefox
  • Resolution set to fixed
  • Status changed from reviewing to closed

comment:4 Changed on 10/19/2016 at 02:16:52 PM by Ross

This is working in Firefox 45, 49, 52 (various headers depending on filters). But Firefox 38 seems to always put a 2.8 header no matter which filters are present.

comment:5 Changed on 10/31/2016 at 09:45:13 AM by trev

  • Cc Ross added

Ross, I cannot reproduce that. In Firefox 38 on macOS 10.12, when I exported filters initially I got "[Adblock Plus 2.0]" header. I then added foobar$websocket filter and got "[Adblock Plus 2.8]" header. Once I removed that filter it was "[Adblock Plus 2.0]" again. Also, I don't really see how this code could depend on the Firefox version. Are you certain that your export didn't have a $websocket filter?

comment:6 Changed on 10/31/2016 at 12:00:53 PM by Ross

Pretty certain as I'd done a fresh install several times and added just a really basic filter. However I can't reproduce it at all in the latest build ( so I'll retest again in a 4X/Nightly version too then verify this. Thanks for checking!

Add Comment

Modify Ticket

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