wired: comma-separated VLAN lists cause false missing-VLAN alerts
@rslater-cs arbeitet bereits daran.
Seit 09.9.2026.
Bewertung
Dieses Issue wurde noch nicht bewertet.
Beschreibung
On CM5 bigbox firmware bigbox-v1-cm-v0.11.2-2026-09-09T110059-v24.10.7, the wired service incorrectly reports missing VLANs and enters degraded state.
The switch’s GE8 uplink reports:
1UP, 4T, 8T, 32T, 100T
However, parse_vlan_membership_string() in src/services/hal/backends/wired/providers/rtl8380m_http.lua:476 returns only VLAN 100.
The parser splits on whitespace, leaving trailing commas on earlier tokens. Its anchored pattern then rejects those tokens.
Impact
False missing-VLAN alerts for admin VLAN 8, user VLAN 32, and WAN VLAN 4 on switch-uplink-cm5. Raw switch data contains all required VLANs; this finding does not establish a forwarding failure.
Reproduction
Observed during a read-only inspection on 2026-09-09. Reproduced using source verified identical to the installed provider by SHA-256:
78f919c72e4151b902e00b0ba07038d8792729efc31288803170029b970f35ba
| Input | Actual VLANs | Expected VLANs |
|---|---|---|
1UP, 4T, 8T, 32T, 100T |
[100] |
[1,4,8,32,100] |
8T, 32T, 100UP |
[100] |
[8,32,100] |
1UP 4T 8T 32T 100T |
[1,4,8,32,100] |
[1,4,8,32,100] |
The first string comes from the GE8 uplink; the second also occurs on external switch ports. The whitespace-only case already parses correctly.
Acceptance criteria
- Parse comma-separated and existing whitespace-separated memberships.
- Preserve tagged, untagged, forbidden, and PVID flags.
- Add regression coverage for both observed comma-separated strings and the whitespace-only case.
- Verify the three false trunk-carriage alerts disappear.
- Vorherrschende Sprache
- Lua
- Sterne
- 1
- Forks
- 0
- Ø Merge
- 5 T. 15 Std.
- Gemergte PRs (30 T.)
- 3
Entwicklungsumgebung
Startet den Dev-Container des Projekts im Browser, mit Ihrem eigenen GitHub-Konto.
- Kein Dockerfile und keine Docker-Compose-Datei
- Keine Pull-Request-Vorlage
- Kein Beitragsleitfaden
Erste Schritte
- Lesen Sie das ganze Issue und danach den Beitragsleitfaden des Projekts.
- Schreiben Sie ins Issue, dass Sie es übernehmen — das erspart doppelte Arbeit.
- Forken Sie das Repository und arbeiten Sie in einem Branch.
- Öffnen Sie einen Pull Request, der die Issue-Nummer nennt.
Mehr aus jangala-dev/devicecode-lua
-
wifi: 40 MHz HE/VHT htmodes unusable — validator only accepts HE40±/VHT40±, which OpenWrt 24.10 ignores (silent 20 MHz)Evtl. vergeben @rslater-cs hat das vor 1 Tag übernommen. Offenbug
Schwierigkeit 2/5 1-3 Stunden Anfängerfreundlichkeit 78/100
jangala-dev/devicecode-lua#344 · 1 zugewiesene Person ·
-
enhancement
Schwierigkeit 5/5 Über eine Woche Anfängerfreundlichkeit 8/100
jangala-dev/devicecode-lua#339 ·
-
Reduce network startup and reconnect delaysEvtl. wieder frei @rslater-cs hat das vor 43 Tagen übernommen, und es ist kein Pull Request offen. Offen
jangala-dev/devicecode-lua#332 · 2 zugewiesene Personen ·
-
Schwierigkeit 4/5 3-5 Tage Anfängerfreundlichkeit 35/100
jangala-dev/devicecode-lua#330 · 1 Kommentar ·
-
Track functional differences from devicecode v0.9.x and devicecode-lua v0.10.6 to v0.11.2Evtl. wieder frei @rslater-cs hat das vor 49 Tagen übernommen, und es ist kein Pull Request offen. Offen
jangala-dev/devicecode-lua#329 · 2 zugewiesene Personen ·
Alle Issues in jangala-dev/devicecode-lua
Ähnliche Issues
-
Wrong step: Ashenvale, Wetlands & Stonetalon Mountains, step 49Evtl. vergeben @b7216309-jpg hat das heute übernommen. Offenguide has-pr
Schwierigkeit 2/5 1-3 Stunden Anfängerfreundlichkeit 68/100
fitzforge/polestar#699 · 2 Kommentare ·
Maintainer antworten meist innerhalb von 1 Tag
-
Schwierigkeit 1/5 Unter einer Stunde Anfängerfreundlichkeit 92/100
LuaLS/lua-language-server#3484 ·
-
bug
Schwierigkeit 2/5 1-3 Stunden Anfängerfreundlichkeit 86/100
lewis6991/gitsigns.nvim#1593 ·
-
Schwierigkeit 2/5 1-3 Stunden Anfängerfreundlichkeit 70/100
latex3/pdf_structure#28 · 4 Kommentare ·
-
code health memtx mvcc
Schwierigkeit 2/5 1-3 Stunden Anfängerfreundlichkeit 78/100
Maintainer antworten meist innerhalb von 1 Tag