Opened on 08/03/2016 at 09:14:36 AM

Closed on 08/03/2016 at 04:40:31 PM

Last modified on 08/16/2016 at 10:36:55 AM

#4300 closed defect (fixed)

Sitekey whitelisting not working for Safari

Reported by: arthur Assignee: kzar
Priority: P2 Milestone: Adblock-Plus-1.12.2-for-Chrome-Opera-Safari
Module: Platform Keywords:
Cc: sebastian, kzar, Ross, scheer Blocked By:
Blocking: Platform: Safari
Ready: yes Confidential: no
Tester: Scheer Verified working: yes
Review URL(s):

Description (last modified by kzar)


OS X 10.11.6
Safari 9.1.2
Adblock Plus devenv built from the latest code (@6a895c3d658d)

These two filters:


How to reproduce

  1. Go to

Observed behaviour

Images aren't shown.

Expected behaviour

Both images should be shown as they are with Chrome.


Attachments (0)

Change History (10)

comment:1 Changed on 08/03/2016 at 11:22:01 AM by kzar

  • Cc kzar added
  • Component changed from Unknown to Platform
  • Description modified (diff)
  • Owner set to kzar
  • Priority changed from Unknown to P2
  • Ready set

Can reproduce, investigating.

comment:2 Changed on 08/03/2016 at 11:24:32 AM by kzar

  • Summary changed from Sitekey whitelisting not working to Sitekey whitelisting not working for Safari

comment:3 Changed on 08/03/2016 at 11:46:19 AM by kzar

  • Description modified (diff)

comment:4 Changed on 08/03/2016 at 12:27:54 PM by kzar

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

Was just a typo, we were sending the filter.addKey message instead of filters.addKey.

comment:5 Changed on 08/03/2016 at 04:38:52 PM by abpbot

A commit referencing this issue has landed:
Issue 4300 - Fix typo in $sitekey adding logic

comment:6 Changed on 08/03/2016 at 04:40:31 PM by kzar

  • Cc Ross scheer added
  • Milestone set to Adblock-Plus-for-Chrome-Opera-Safari-next
  • Resolution set to fixed
  • Status changed from reviewing to closed

comment:7 Changed on 08/16/2016 at 09:20:15 AM by scheer

@kzar - Sorry, but to be clear, the two images are one of a cat and one of the ABP logo?


comment:8 Changed on 08/16/2016 at 09:50:59 AM by kzar

Yep, the expected behaviour is that both images are shown, both the cat and our logo. (The same as if you try it with Chrome.)

comment:9 Changed on 08/16/2016 at 10:10:12 AM by scheer

  • Tester changed from Unknown to Scheer
  • Verified working set
  • Adding the filter whilst all other filter list subscriptions are disabled allows the two images to load as intended.

OS X 10.11.6
Safari 9.1.2

comment:10 Changed on 08/16/2016 at 10:36:55 AM by sebastian

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 kzar.
Note: See TracTickets for help on using tickets.