Open 0f-0b opened 1 day ago
@stephenhawk8054 The latest syntax for ipaddress=
values which are not regex is "starts with", so ::
ended up matching all IPv6 addresses starting with ::
, which is a lot. For now I converted the value to a regex.
I chose the "starts with" approach because this makes it more convenient for IPv4 addresses, but obviously it's not working well for IPv6 addresses. We will need to agree on a syntax which is a good balance between convenience while keeping the implementation straightforward.
Ah right. I forgot ipv6 has more problems with startsWith
. Actually ipv4 can have issue too, 149.34.253.24
can end up matching more than expected. And might end up in using regex a lot.
Maybe a hint like
ipaddress=startsWith ::,domain=
ipaddress=startsWith 149.34.253.24,domain=
to signal using startsWith
. If there's no =startsWith
, match exactly?
I will go with wildcard at the end, i.e. ::*
for anything starting with ::
, and ::
for exact match.
Wildcard is a good idea 👍
Small note: the above commit has been forgotten to be mentioned in release changelog
I don't mention commits which are not of interest to stable release. ipaddress=
didn't exist in previous stable release, so a fix/change to an newly introduced option not yet in stable is of no interest. People who want to know all the details can browse all commits since last stable release.
Prerequisites
I tried to reproduce the issue when...
Description
The “uBlock filters – Privacy” list contains the following filter, intended to prevent access to the unspecified address
::/128
.However, IPv4-mapped addresses
::ffff:0:0/96
also happen to be blocked by this filter.A specific URL where the issue occurs.
Steps to Reproduce
https://[::ffff:14cd:f3a6]/
.Expected behavior
The page loads. The browser may warn about the certificate being invalid.
Actual behavior
The page is blocked by this extension.
uBO version
1.59.1rc1
Browser name and version
Firefox 131.0b7
Operating System and version
macOS 14.6.1