Falsche IPv6-Syntax in Hoodfile #27

New Issue

fbl · 2021-01-04T13:18:30+01:00

fbl commented

2021-01-04 13:18:30 +01:00

Aktuell ist im KeyXchange das Prefix im Hoodfile falsch angegeben: fd43:5602:29bd:abcd:/64, korrekt müsste es fd43:5602:29bd:abcd::/64 (mit zwei ::) heißen.

Das müsste der Fall sein, weil die Firmware dieses Prefix nicht korrekt parsed. Dies sollte repariert werden, um fehlerhafte Einträge zu verhindern. (Denn niemand erwartet, dass er ein IPv6 Prefix "falsch" angeben muss, damit es funktioniert)

Aktuell ist im KeyXchange das Prefix im Hoodfile falsch angegeben: `fd43:5602:29bd:abcd:/64`, korrekt müsste es `fd43:5602:29bd:abcd::/64` (mit zwei `::`) heißen. Das müsste der Fall sein, weil die Firmware dieses Prefix nicht korrekt parsed. Dies sollte repariert werden, um fehlerhafte Einträge zu verhindern. (Denn niemand erwartet, dass er ein IPv6 Prefix "falsch" angeben muss, damit es funktioniert)

fbl added the

bug

node

labels 2021-01-04 13:18:46 +01:00

adschm commented

2021-01-04 13:40:18 +01:00

Man hat sich damals für das falsche Prefix entscheiden, weil das Zusammensetzen der Adressen mit dem syntaktisch richtigen erheblich komplizierter ausgefallen wäre.

fbl commented

2021-01-04 15:41:10 +01:00

Das kann ich jetzt nicht ganz nachvollziehen. Im Zweifel würde es ja schon genügen einfach den extra Doppelpunkt mit einem Regex zu entfernen?

Aber es ist ja an sich auch egal, wie das in die Firmware gekommen ist. Auf jeden Fall muss es weg. ;-)

Das kann ich jetzt nicht ganz nachvollziehen. Im Zweifel würde es ja schon genügen einfach den extra Doppelpunkt mit einem Regex zu entfernen? Aber es ist ja an sich auch egal, wie das in die Firmware gekommen ist. Auf jeden Fall muss es weg. ;-)

adschm commented

2021-01-04 15:46:57 +01:00

Den extra Doppelpunkt zu entfernen funktioniert nur unter der Prämisse, das man am Ende zwei Doppelpunkte hat (und keine andere valide Notation für ein /64 oder ggf. ein Netz mit Nullern auf dem vierten Block etc.). Wenn man mit validen Prefixen arbeitet, muss man hier eine Fallunterscheidung für das Zusammensetzen der Adresse machen.

Ist aber im Prinzip müßig, das hier theoretisch zu diskutieren, du kannst ja einfach mal versuchen, einen Patch für configurenetwork zu bauen. Das Hauptproblem wird eher, wie man hier die Kompatibilität darstellt (ggf. muss man dann noch einen Konverter für php bauen, der das Prefix in beiden Syntax-Versionen anbietet).

Den extra Doppelpunkt zu entfernen funktioniert nur unter der Prämisse, das man am Ende zwei Doppelpunkte hat (und keine andere valide Notation für ein /64 oder ggf. ein Netz mit Nullern auf dem vierten Block etc.). Wenn man mit validen Prefixen arbeitet, muss man hier eine Fallunterscheidung für das Zusammensetzen der Adresse machen. Ist aber im Prinzip müßig, das hier theoretisch zu diskutieren, du kannst ja einfach mal versuchen, einen Patch für configurenetwork zu bauen. Das Hauptproblem wird eher, wie man hier die Kompatibilität darstellt (ggf. muss man dann noch einen Konverter für php bauen, der das Prefix in beiden Syntax-Versionen anbietet).

fbl added this to the 20210211-beta milestone 2021-01-05 16:05:03 +01:00

fbl referenced a pull request that will close this issue

2021-01-09 00:01:16 +01:00

fff-network: Fix incorrect IPv6 syntax in functions #69

fbl commented

2021-01-27 12:41:37 +01:00

Das ganze soll in Zukunft mit der Kombination aus #77 und #90 gelöst werden.

fbl self-assigned this 2021-01-27 12:41:47 +01:00

fbl commented

2021-01-31 14:57:29 +01:00

Fixed with afe633e874

Fixed with afe633e874b4cd8c8f723d5999d126ec7e1cdc1e