V6: after migration "Domains to be excluded from Top Domain Lists and Query Log" isn't excluded from "Top Domains"

[thomasmerz]

Versions

Core version is v6.0.4 (Latest: v6.0.4)
Web version is v6.0.1 (Latest: v6.0.1)
FTL version is v6.0.2 (Latest: v6.0.2)

Platform

• OS and version: Docker
• Platform: Docker

Expected behavior

After migration to V6 "Top Permitted Domains" and "Top Blocked Domains" does show "Domains to be excluded from Top Domain Lists and Query Log".

Actual behavior / bug

"Top Permitted Domains" and "Top Blocked Domains" should NOT show all "Domains to be excluded from Top Domain Lists and Query Log".

Steps to reproduce

See screenshots:

Screenshots

After "fixing" ^*\.syncthing\.net$ to syncthing.net it doesn't show up anymore.

Is this an error while migration or with the regex?

[DL6ER]

At first I was a bit confused but I think you just mixed the texts under "Expected behavior" and "Actual behavior / bug". All good.

After "fixing" ^*\.syncthing\.net$ to syncthing.net it doesn't show up anymore.

Is this an error while migration or with the regex?

The regex is wrong. ^* doesn't really mean anything, the ^* should probably not be there. I guess the string was *.syncthing.net before the migration?

[DL6ER]

Should be fixed by #2311

[thomasmerz]

At first I was a bit confused but I think you just mixed the texts under "Expected behavior" and "Actual behavior / bug". All good.
…
The regex is wrong. ^* doesn't really mean anything, the ^* should probably not be there. I guess the string was *.syncthing.net before the migration?

Yes, I mixed the behaviours 😆 and yes, the "string was *.syncthing.net before the migration" 👍🏼 Thanks for fixing, waiting for release…