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

Test #2

Open
wants to merge 1 commit into
base: master
Choose a base branch
from
Open

Test #2

wants to merge 1 commit into from

Conversation

galeksandrp
Copy link
Owner

@galeksandrp galeksandrp commented Jul 14, 2016

Since default_off is not accepted now, the only cause of ruleset disabled by default is platform="mixedcontent". I'd prefer not cause some mixed content cause platform="mixedcontent" such as content blocked by Ublock Origin. Or at least by EasyList.

As well as https://www.w3.org/TR/mixed-content/#category-optionally-blockable

  • <img>
  • <video>
  • <audio>
  • <source>

As well as

  • <form>

since it not blocked by default in Chrome 56.0.2924.87.

I'm also used/using bramus/mixed-content-scan@master --depth=1 on all domains of rulesets of all my PRs.

<br>https://[^<]* <a>\?</a><div>(?:http://[^<]*\.(?:png|jpg|jpeg|gif)(?:(?:\?|#).*)?(?:<br>)?)+</div>

galeksandrp pushed a commit that referenced this pull request Apr 25, 2017
galeksandrp pushed a commit that referenced this pull request May 23, 2017
* [Nature.com] Remove wildcard and update subdomains #1

* [Nature.com] Remove wildcard and update subdomains #2

* [Nature.com] Add subdomains

* [Nature.com] Add missing target

* [Nature.com] Fix medicine related subdomains

* [Nature.com] Remove (partial)

* [Nature.com] Add target for verify.nature.com

* Rename Nature.xml to Nature.com.xml

* [Nature.com] Remove data.nature.com

* Update Nature.com.xml

* [Nature.com] Add my.nature.com

* [Nature.com] Remove my.nature.com

* [Nature.com] Add my.nature.com

* [Nature.com] Remove invalid target
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant