wired: comma-separated VLAN lists cause false missing-VLAN alerts
@rslater-cs ya está trabajando en esto.
Desde el 9/9/2026.
Evaluación
Este issue todavía no se ha evaluado.
Descripción
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.
- Lenguaje dominante
- Lua
- Estrellas
- 1
- Forks
- 0
- Merge medio
- 5 d 15 h
- PR fusionados (30 d)
- 3
Preparar el entorno
Inicia el contenedor de desarrollo del proyecto en tu navegador, con tu propia cuenta de GitHub.
- Sin Dockerfile ni archivo de Docker Compose
- Sin plantilla de pull request
- Sin guía de contribución
Primeros pasos
- Lee el issue completo y luego la guía de contribución del proyecto.
- Comenta en el issue que vas a ocuparte — evita que dos personas hagan lo mismo.
- Haz un fork del repositorio y trabaja en una rama.
- Abre un pull request que haga referencia al número del issue.
Más de jangala-dev/devicecode-lua
-
bug
Dificultad 2/5 1-3 horas Aptitud para principiantes 78/100
jangala-dev/devicecode-lua#344 ·
-
enhancement
Dificultad 5/5 Más de una semana Aptitud para principiantes 8/100
jangala-dev/devicecode-lua#339 ·
-
Reduce network startup and reconnect delaysQuizá libre de nuevo @rslater-cs la tomó hace 42 días y no hay ningún pull request abierto. Abierto
jangala-dev/devicecode-lua#332 · 2 asignados ·
-
Dificultad 4/5 3-5 días Aptitud para principiantes 35/100
jangala-dev/devicecode-lua#330 · 1 comentario ·
-
Track functional differences from devicecode v0.9.x and devicecode-lua v0.10.6 to v0.11.2Quizá libre de nuevo @rslater-cs la tomó hace 48 días y no hay ningún pull request abierto. Abierto
jangala-dev/devicecode-lua#329 · 2 asignados ·
Todos los issues de jangala-dev/devicecode-lua
Issues similares
-
Dificultad 2/5 1-3 horas Aptitud para principiantes 78/100
Los mantenedores suelen responder en 2 días
-
Dificultad 2/5 1-3 horas Aptitud para principiantes 78/100
luainkernel/lunatik#2018 ·
Los mantenedores suelen responder en 1 día
-
Dificultad 1/5 Menos de una hora Aptitud para principiantes 86/100
LuaLS/lua-language-server#3481 ·
-
Kill Ishamuhale for Ishamuhale's FangPosiblemente ocupada Un pull request vinculado a esta issue está abierto o ya se fusionó. Abiertobug discord guide:04-the-barrens locale:enUS triage v0.2.6
Dificultad 2/5 1-3 horas Aptitud para principiantes 64/100
Los mantenedores suelen responder en 1 día
-
Dificultad 1/5 Menos de una hora Aptitud para principiantes 78/100