Skip to content
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

uplink/notunnel: use label-mac as unique part of ffuplinks-dhcpname #783

Open
SvenRoederer opened this issue Apr 10, 2020 · 5 comments
Open

Comments

@SvenRoederer
Copy link
Contributor

in freifunk-berlin/firmware-packages@8e2b55e we use the random MAC-address to have a 2nd unique name when registering in DHCP-server.
in OpenWrt a function to retrieve the MAC-address printed on the outside label was added (https://git.openwrt.org/?p=openwrt/openwrt.git;a=commit;h=0340718863040919ecaba74b2173ed52ebc194ce - after OpenWrt-19.07 branch of).
Let's use this address in the uplinks hostname.

So the host can easily looked up in the upstream-LAN, esp. when more than one Router is installed.

@pmelange
Copy link
Contributor

This is an interesting idea. Buy why not just go with something human readable like a concatenation of the hostname and "ffuplink"?

I assume you mean setting the value for network.ffuplink.hostname

@Akira25
Copy link
Member

Akira25 commented Apr 13, 2020

I'd also prefer a solution which is human-readable without further need to lookup something. Unless there is a good point, why the ffuplink-hostname needs to be machine readable.

@SvenRoederer
Copy link
Contributor Author

The point is: when the defaults-script is running the hostname is still "gib-mir-einen-namen". This will also not be unique and also will cause name-clashes.

@SvenRoederer
Copy link
Contributor Author

Btw: You are aware, what we use at the moment? The currently used random string can't even be lookup easily, beside checking the config of all connected routers.

@pmelange
Copy link
Contributor

Well, I'm still of the opinion that ffuplink should be disabled until after the router runs through the wizard. See #603

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

3 participants