-
Notifications
You must be signed in to change notification settings - Fork 325
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
UniFi 6 LR status page - internal server error 500 #3020
Comments
Related commits from Oct. 14th 2022: I assume, we can use |
@neocturne what do you think, editing the match-regex to include wl* or using something like get_board_phy_name() method? |
I would like to understand where this name change comes from first. Is hostapd or some related script doing this? Can we be sure that all PHYs are always consistently renamed? Can we have a |
hm, @Djfe linked the commits, did you see them? |
Can't reproduce with a v1. This image is from the Gluon CI. @Djfe Please build an image with the minimal-site from the Gluon repo and see if you can reproduce the problem. As this issue does not affect devices already supported by Gluon (This ticket is for a not-yet supported revision), I remove it from the milestone and would like to close it in the next days if no objections arise.
|
Seems like the interface names are nowadays taken from the board.json: This is enabled for some devices mt7622
I think this issue has also been seen on a NETGEAR WAX206. |
Executing a script with:
returns
So I think that moving to UPDATE: Well I was wrong - get_board_phy_name does not return anything for devices which do not have the wifi information in the board.json :( Why can't we just use something like |
Just replaced my node at home with a Xiaomi AX3200, it seems to be affected as well. |
Bug report
What is the problem?
the status page on a UniFi 6 LR errors like this (probably affects all version, this one is a v3, PR pending)
see http://[2a03:2260:3006:115:6222:32ff:fe6a:37d5]/cgi-bin/status
What is the expected behaviour?
Status page should render
Gluon Version:
1b6ddac (master 2 weeks ago)
Site Configuration:
https://github.com/ffac/site/tree/v2023.2.0-5
Custom patches:
quite a few, but none affecting gluon-core
https://github.com/ffac/site/tree/v2023.2.0-5/patches
I started debugging the issue, line 26 is effectively this one:
gluon/package/gluon-core/luasrc/usr/lib/lua/gluon/wireless.lua
Line 40 in a137019
The issue appears to stem from a line above:
local phy_idx = v:match('^phy(%d+)$')
If I understand it correctly, it expects wiphys to contain
phy
in their name in order to extract the correct phy_idx.This device has (atleast since OpenWrt 23) the phy names
wl0
andwl1
The text was updated successfully, but these errors were encountered: