wired: comma-separated VLAN lists cause false missing-VLAN alerts
@rslater-cs đang làm issue này rồi.
Từ ngày 9/9/2026.
Đánh giá
Issue này chưa được đánh giá.
Mô tả
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.
- Ngôn ngữ chính
- Lua
- Star
- 1
- Fork
- 0
- Merge trung bình
- 5 ngày 15 giờ
- Pull request đã merge (30 ngày)
- 3
Chuẩn bị môi trường
Khởi chạy dev container của dự án ngay trên trình duyệt, bằng tài khoản GitHub của bạn.
- Không có Dockerfile hay tệp Docker Compose
- Không có mẫu pull request
- Không có hướng dẫn đóng góp
Bắt đầu từ đâu
- Đọc hết issue, rồi đọc hướng dẫn đóng góp của dự án.
- Bình luận trên issue rằng bạn sẽ nhận — tránh hai người làm cùng một việc.
- Fork repository và làm thay đổi trên một nhánh.
- Mở pull request có tham chiếu số hiệu của issue.
Issue khác của jangala-dev/devicecode-lua
-
wifi: 40 MHz HE/VHT htmodes unusable — validator only accepts HE40±/VHT40±, which OpenWrt 24.10 ignores (silent 20 MHz)Có thể đã có người làm @rslater-cs đã nhận hôm nay. Đang mởbug
Độ khó 2/5 1-3 giờ Mức phù hợp với người mới 78/100
jangala-dev/devicecode-lua#344 · 1 người được giao ·
-
enhancement
Độ khó 5/5 Hơn một tuần Mức phù hợp với người mới 8/100
jangala-dev/devicecode-lua#339 ·
-
Reduce network startup and reconnect delaysCó thể làm lại được @rslater-cs đã nhận 42 ngày trước và không có pull request nào đang mở. Đang mở
jangala-dev/devicecode-lua#332 · 2 người được giao ·
-
Độ khó 4/5 3-5 ngày Mức phù hợp với người mới 35/100
jangala-dev/devicecode-lua#330 · 1 bình luận ·
-
Track functional differences from devicecode v0.9.x and devicecode-lua v0.10.6 to v0.11.2Có thể làm lại được @rslater-cs đã nhận 48 ngày trước và không có pull request nào đang mở. Đang mở
jangala-dev/devicecode-lua#329 · 2 người được giao ·
Tất cả issue của jangala-dev/devicecode-lua
Issue tương tự
-
guide
Độ khó 2/5 1-3 giờ Mức phù hợp với người mới 72/100
fitzforge/polestar#672 · 1 bình luận ·
Maintainer thường phản hồi trong vòng 1 ngày
-
code health memtx mvcc
Độ khó 2/5 1-3 giờ Mức phù hợp với người mới 78/100
Maintainer thường phản hồi trong vòng 1 ngày
-
Độ khó 2/5 1-3 giờ Mức phù hợp với người mới 82/100
Maintainer thường phản hồi trong vòng 1 ngày
-
Witch 2Đang mở
Độ khó 1/5 Dưới một giờ Mức phù hợp với người mới 91/100
LuaLS/lua-language-server#3482 ·
-
Độ khó 2/5 1-3 giờ Mức phù hợp với người mới 78/100
Maintainer thường phản hồi trong vòng 2 ngày