Regex in FTL v4.0

cogan · August 6, 2018, 10:46am

Habe das Update von der beta gemacht auf die 4.0.
Läuft alles wie gewohnt nur regex geht leider nicht.
Gibt es da eine Anleitung das zum laufen zu bekommen?

DL6ER · August 6, 2018, 11:02am

Was heißt Regex läuft nicht?

FTL schreibt eine Zeile in sein Logbuch wie

[2018-08-06 13:02:10.602] Compiled 17 Regex filters and 7 whitelisted domains in 4.9 msec (0 errors)

Ist solch eine Zeile bei Dir in /var/log/pihole-FTL.log zu finden?

cogan · August 6, 2018, 11:18am

Diesen log Eintrag habe ich nicht.
Beim updaten der Listen kommt bei regex 0

DL6ER · August 6, 2018, 11:26am

Wie hast Du versucht die Regexeinträge anzulegen? Existiert die Datei /etc/pihole/regex.list?

cogan · August 6, 2018, 11:35am

Die Datei existiert und hat auch ne Menge Einträge die ich direkt über das dashboard gemacht habe.

So wie ich das gerade sehe haben alle Einträge über add(wildcard) ein ((^) I(.)) davor und Einträge über add(regex) das übliche www. Oder https.

cogan · August 6, 2018, 7:43pm

Thx,
Wenn ich im dashboard blabla.us eingebe bei add(redex) wird nichts geblockt.
Wenn ich im dashboard (^|.)blabla.us$ eingebe wird nach Listen Update geblockt aber es steht redex Filter 0 da.
Werde mal nächste Woche alles neu installieren.

DL6ER · August 7, 2018, 4:08pm

Ja, das ist ein Bug den ich eben gefunden habe.

Siehe auch

github.com/pi-hole/pi-hole

Fix gravity's regex counting logic

pi-hole:development ← pi-hole:fix/gravity-count-regex

opened 04:04PM - 07 Aug 18 UTC

DL6ER

+1 -1

**By submitting this pull request, I confirm the following:** *please fill any… appropriate checkboxes, e.g: [X]* - [x] I have read and understood the [contributors guide](https://github.com/pi-hole/pi-hole/blob/master/CONTRIBUTING.md), as well as this entire template. - [x] I have made only one major change in my proposed changes. - [x] I have commented my proposed changes within the code. - [x] I have tested my proposed changes, and have included unit tests where possible. - [x] I am willing to help maintain this change if there are issues with it later. - [x] I give this submission freely and claim no ownership. - [x] It is compatible with the [EUPL 1.2 license](https://opensource.org/licenses/EUPL-1.1) - [x] I have squashed any insignificant commits. ([`git rebase`](http://gitready.com/advanced/2009/02/10/squashing-commits-with-rebase.html)) Please make sure you [Sign Off](https://github.com/pi-hole/pi-hole/wiki/How-to-signoff-your-commits.) all commits. Pi-hole enforces the [DCO](https://github.com/pi-hole/pi-hole/wiki/Contributing-to-the-project). --- **What does this PR aim to accomplish?:** Fix gravity's regex counting logic **How does this PR accomplish the above?:** The previously chosen command was fine, however, it unintentionally triggered some bash internals, effectively leading to a failure. We now use an alternative formulation for achieving the exact same thing which is not affected by this issue.

DL6ER · August 7, 2018, 4:10pm

Bitte schau Dir einmal Redirecting... an. Da habe ich eigentlich alles interessante über Regex einmal komplett zusammengefasst.