[0/2] zonconf: replace the nic-names with eth... and wlan... or not?

Message ID 20190903214624.2280-1-ipfire@starkstromkonsument.de


Alexander Koch 3 Sep 2019, 9:46 p.m. UTC

I stumbled into this when I was searching for an error in my vlan and bridge-config and want to start a discussion about if this should be changed or not.

The zoneconf.cgi contains the following code-snippet:

> # Name the physical NICs
> # Even though they may not be really named like this, we will name them ethX or wlanX
> my $ethcount = 0;
> my $wlancount = 0;
> foreach (@nics) {
> 	my $nic = $_->[1];
> 	if (-e "/sys/class/net/$nic/wireless") {
> 		$_->[1] = "wlan$wlancount";
> 		$_->[2] = 1;
> 		$wlancount++;
> 	} else {
> 		$_->[1] = "eth$ethcount";
> 		$ethcount++;
> 	}
> }

This renames all the nics to ethX or wlanX for the WUI. When you use any commandline tool (e.g ip addr or brctl show) the names of the NICs will mismatch with the ones shown in the WUI. Same for the assigened VLAN-IDs.

For example, one of my setups:

- WUI name --> real name
- eth0 --> red0
- eth1 --> eth1
- eth2 --> orange0
- wlan0 --> blue0
- wlan1 --> wlan1

What is the goal of doing this? I personally think this is more confusing than useful. I would prefer to not rename the NICs. What do you think?

I attached a patch to change the naming back to the real names. Please decide to merge it or not, depending on where the discussion ends.

Secondly I created a patch to add the current output of "brctl show" and "ip addr" to the zoneconfig page. It's quite useful to have this by the hand without having to open a shell sometimes.

Regards, Alex

Alex Koch (2):
  zoneconf: Show real names of NICs in the WUI
  zoneconf: Add status output of "brctl show" and "ip addr"

 html/cgi-bin/zoneconf.cgi | 34 ++++++++++++++++++++++++----------
 langs/de/cgi-bin/de.pl    |  5 ++++-
 langs/en/cgi-bin/en.pl    |  5 ++++-
 3 files changed, 32 insertions(+), 12 deletions(-)