Skip to content

Commit

Permalink
address #2520 (comment)
Browse files Browse the repository at this point in the history
  • Loading branch information
@okiehsch
okiehsch committed Jun 11, 2018
1 parent 4682c98 commit 2bec415
Showing 1 changed file with 2 additions and 2 deletions.
4 changes: 2 additions & 2 deletions filters/filters.txt
View file
Original file line number Diff line number Diff line change
Expand Up @@ -2640,8 +2640,8 @@ mylink.nu,mylink.st##a[href^="https://go.nordvpn.net/"]
cookiesflix.com##script:inject(addEventListener-defuser.js, load, adsense)

! https://github.com/uBlockOrigin/uAssets/issues/775
postimg.org##^script:has-text(676574456c656d656e747342795461674e616d65)
||piguiqproxy.com^$xmlhttprequest,redirect=nooptext,domain=postimg.org
! https://github.com/uBlockOrigin/uAssets/issues/2520#issuecomment-396112781
postimg.*##script:inject(abort-current-inline-script.js, atob, null,f)

! Uponit
! https://github.com/uBlockOrigin/uAssets/issues/259
Expand Down

5 comments on commit 2bec415

@jspenguin2017
Copy link
Contributor

@jspenguin2017 jspenguin2017 commented on 2bec415 Jun 12, 2018 •
edited
Loading

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@okiehsch abort-current-inline-script.js only accepts 2 arguments. If you pass more arguments than the scriptlet accepts, the extra ones would be ignored.

Note that you cannot pass null,f as an argument because , is a special character and uBO will treat it as argument separator.

@okiehsch
Copy link
Contributor Author

@okiehsch okiehsch commented on 2bec415 Jun 12, 2018 •
edited
Loading

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Oh, I totally overlooked that the , means a new argument.

@jspenguin2017
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Also @gorhill FYI.

Should there be a way to escape ,?

@okiehsch
Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

3d98a88

@jspenguin2017
Copy link
Contributor

@jspenguin2017 jspenguin2017 commented on 2bec415 Jun 12, 2018 •
edited
Loading

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Actually, I found a way to escape it. Only works if the argument accepts RegExp.
image

Please sign in to comment.