Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

CSP change is not applied to iframes #440

Closed
Alex-302 opened this issue Nov 24, 2016 · 1 comment
Closed

CSP change is not applied to iframes #440

Alex-302 opened this issue Nov 24, 2016 · 1 comment
Assignees
Labels
Bug
Milestone

Comments

@Alex-302
Copy link
Member

@Alex-302 Alex-302 commented Nov 24, 2016

http://zserials.tv/zarubezhnye/airwolf.php
player #2 (share-serials.net)
Browser: Chrome
Screenshot
http://img.ii4.ru/images/2016/11/24/795372_kiss_158kb.png
the rule blocks WS only on the main page
|ws*://*$domain=share-serials.net|zserials.tv

This ad is locked by JS rule(added to Russian filter)

share-serials.net,share-serials.tv#%#window.Worker = function() { this.postMessage = function() {} };

it should be whitelisted to do the check:

share-serials.net,share-serials.tv#@%#window.Worker = function() { this.postMessage = function() {} };
@Alex-302 Alex-302 added the Bug label Nov 24, 2016
@ameshkov ameshkov changed the title [Chrome] Cannot block WS in iframe CSP change is not applied to iframes Nov 24, 2016
@Alex-302

This comment has been minimized.

Copy link
Member Author

@Alex-302 Alex-302 commented Nov 24, 2016

The problem is more wide.
More examples where WS are not blocked not in iframes only:
http://hdkinohit.net/serials/6051-zavtra-ne-nastupit-1-sezon-2016.html
http://www.24video.sex/jsecUser/create
http://ostfilmtv.org/
Previous beta and release version blocks WS on these sites(except 24video.sex)

Mizzick added a commit that referenced this issue Nov 25, 2016
Mizzick added a commit that referenced this issue Nov 25, 2016
Mizzick added a commit that referenced this issue Nov 25, 2016
@Mizzick Mizzick closed this Nov 25, 2016
ameshkov added a commit that referenced this issue Nov 25, 2016
Alex-302 added a commit to AdguardTeam/AdguardFilters that referenced this issue Dec 1, 2016
maximtop pushed a commit that referenced this issue Apr 16, 2019
… to master

* commit '1e3f459b624bfc8b3634957d6e0264aac2856317':
  fix/1354 ignore hostnames which length is less or equal to 8
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
3 participants
You can’t perform that action at this time.