libremesh / lime-packages

LibreMesh packages configuring OpenWrt for wireless mesh networking
https://libremesh.org/
GNU Affero General Public License v3.0
277 stars 96 forks source link

limeWeb has error :error on line 226 at column 1: Extra content at the end of the document #897

Open ggsnpdsn opened 3 years ago

ggsnpdsn commented 3 years ago

Use http://10.163.166.176/cgi-bin/luci/lime/essentials to access router, an error occurs: This page contains the following errors: error on line 226 at column 1: Extra content at the end of the document Below is a rendering of the page up to the first error.

/etc/config/lime: config lime 'wifi' option ap_ssid 'ap_ssid'

config lime 'network' option main_ipv4_address '10.%N1.0.0/16' option main_ipv6_address 'fd%N1:%N2%N3:%N4%N5::/64'

config lime 'system' option hostname 'LiMe-%M4%M5%M6'

config libremap 'settings'

config libremap 'location'

config libremap 'contact'

ilario commented 3 years ago

lime-webui has been abandoned since long time, in favour of lime-app. Can you try if lime-app works for your needs? Nevertheless, I think there are people out there still using lime-web. Any help for solving this issue is welcome!

ggsnpdsn commented 3 years ago

Sometimes, I want to modify the channel or transmit power and other parameters, but the lime-app cannot modify it.Is it possible to add some configuration parameters in lime-app, such as channel, power, encryption password, distance and other basic parameters

ilario commented 3 years ago

I agree (see https://github.com/libremesh/lime-packages/issues/327). We need @germanferrero @spiccinini @patacon13 @nicopace opinion here.

ilario commented 2 years ago

TL;DR: the fix would be to rewrite a lot of stuff

I had a look at what is failing: this line https://github.com/libremesh/lime-packages/blob/25c6ebeb027a9e85cdae3c8004b1d4a0bee48df6/packages/lime-webui/luasrc/model/essentials.lua#L90 tries to get the distance parameter that nowadays has been splitted in distance_2ghz and distance_5ghz. But even replacing the distance by one of the other two valid values, the page stays blank (but the error is not there anymore). In the source code of the essentials page, there are plenty of references to LibreMap, which is defunct since long time. So it looks hard to maintain now and would require rewriting a lot of the lime-webui code or at least of the essentials page. I would expect no effort in this direction from the core devs, but I'm sure that any contribution fixing things will be welcome anyway.

Even if the error is confirmed, I suggest to close this issue, unless anyone is willing to work on it. Indeed, we could even remove the lime-webui package from this repository.