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

Support multiple client ESSIDs #474

Open
jplitza opened this issue Sep 2, 2015 · 11 comments
Open

Support multiple client ESSIDs #474

jplitza opened this issue Sep 2, 2015 · 11 comments
Labels
0. type: enhancement The changeset is an enhancement

Comments

@jplitza
Copy link
Member

jplitza commented Sep 2, 2015

I think there are valid reasons for a community (not a single node owner) to have multiple ESSIDs for clients on their nodes, be it they want to announce a local "<community>.freifunk.net" as well as the currently advocated "Freifunk", or be it that their usual ESSID is dual-band but they want to have another one only for 5 GHz.

Since the last change, the site.conf reads:

wifiX = { ap = { ssid = "foo" }, ... }

This could easily be changed to either of the following:

wifiX = { ap = [ "foo", "bar" ], ... }
wifiX = { ap = [ { ssid = "foo" }, { ssid = "bar" } ], ... }

So questions are: Is this a feature that would be merged, and which of the site.conf formats is preferred? Also, what implications if any does this have on airtime consumption?

@FreifunkUFO
Copy link

if we would have more ESSIDs it would be nice to have them in seperate network-enviroments (different ips, subnet ...) . so config would be more complex :-o
btw: in leipzig our meshkit-firmware we are running 2 batman-adv networks on each node (with 2 essids, separated by wifi-vlan), its nice. atm our compatible gluon-firmware is only meshing with one of them.

@FreifunkUFO
Copy link

siehe auch #458 und weitere dort verlinkte ähnlichen tickets..

@jplitza
Copy link
Member Author

jplitza commented Sep 3, 2015

#458 is about something completely different, namely configuring more ESSIDs in Luci, not via the site.conf as I'd like to have it.

And having separated networks also is something far more complicated and thus not in the scope of what I proposed. You need not only different ESSIDs for the clients, but also different IBSS/802.11s connections, fastd tunnels and batman interfaces.

@A-Kasper
Copy link
Contributor

A-Kasper commented Sep 3, 2015

If I'm right, this is the most frequent requested feature. So I really
would suggest to implement it.
It would be nice to merge these ideas:
Setting multiple ESSIDs in the site.conf and add an option "configurable"
to an additional ESSID to make it free configurable over Luci.

2015-09-03 13:08 GMT+02:00 Jan-Philipp Litza notifications@github.com:

#458 #458 is about
something completely different, namely configuring more ESSIDs in Luci,
not via the site.conf as I'd like to have it.

And having separated networks also is something far more complicated and
thus not in the scope of what I proposed. You need not only different
ESSIDs for the clients, but also different IBSS/802.11s connections, fastd
tunnels and batman interfaces.


Reply to this email directly or view it on GitHub
#474 (comment)
.

@jplitza
Copy link
Member Author

jplitza commented Sep 3, 2015

Do you propose to have a fixed SSID and en- or disable it in Luci? I can't think of any usecase for it, but I'll happily learn one. Or do you propose to have a freely configurable SSID in Luci? That simply won't happen upstream, no matter how often it is requested.

@wusel42
Copy link

wusel42 commented Sep 18, 2015

Question: are all currently supported devices capable of doing up to three AP (2 open ones for Freifunk, one secured one for private WiFi) alongside ad-hoc and/or 802.11s? Can this be detected at run time, so e. g. private WiFi isn't offered when two Feifunk SSIDs are to be used and two is the limit?

@jplitza
Copy link
Member Author

jplitza commented Sep 19, 2015

Obviously I cannot answer that question definitely for every supported device, but I am pretty sure the answer is "yes".

You can list the configurations supported by your chip by running iw phy and looking for "valid interface combinations". For TP-Link devices, those look like

* #{ managed, WDS } <= 2048, #{ AP, mesh point } <= 8, #{ P2P-client, P2P-GO } <= 1, #{ IBSS } <= 1,

meaning that we can have up to 8 AP or mesh interfaces, independent of the IBSS interface. So even if private WiFi is enabled and the community uses 802.11s (each eating one of those cells), the community could still use up to 6 client ESSIDs.

I can only speculate what the supported combinations for other devices are, but I am pretty sure every supported device supports more than two {AP, mesh point} interfaces. The other way round, we already require the devices to support AP and mesh point simultaneously.


Now a whole lot of people have commented here, but I still would like to hear word from @NeoRaider and @tcatm if this could get merged if I implemented it. So... ping?

@neocturne
Copy link
Member

I'd like to avoid supporting multiple ESSIDs.

The only usecase I've heard so far (a unified "Freifunk" ESSID) is inherently broken as it completely breaks roaming in "border regions" where nodes of multiple communities are nearby. Please re-read the definition of a ESSID if you think this might be a good idea.

@wusel42
Copy link

wusel42 commented Sep 23, 2015

There's one valid use case I think: migrating away from an old to a new SSID setting without kicking hundreds of users over night (e. g. »guetersloh.freifunk.net« => (possibly) »gt.freifunk.net«, to accomodate those parts of an administrative district that don't like to be summarized under it's lead city's name). See also Münster, Göttingen, ...

I totally agree with the argument against one unified ESSID; but that one is used anyway already, despite being broken by design.

@jplitza
Copy link
Member Author

jplitza commented Sep 29, 2015

The only usecase I've heard so far (a unified "Freifunk" ESSID) is inherently broken as it completely breaks roaming in "border regions" where nodes of multiple communities are nearby. Please re-read the definition of a ESSID if you think this might be a good idea.

What about the one I mentioned in my first post, namely having a separate SSID for 5 GHz (to make it possible for devices that support it to force 5 GHz) while still having a unified one in 2.4 GHz as well as 5 GHz, so that 5 GHz is used at least sometimes?

Also, I know the definition of an ESSID. And that in the Freifunk context, it often simply does not apply. As far as I know, none of the currently existing Layer 3 mesh protocols support roaming (which is why @tcatm is developing it), and thus all communities not using B.A.T.M.A.N. advanced shouldn't use a single ESSID according to your argument.

@bittorf
Copy link

bittorf commented Sep 30, 2015

@jplitza we support roaming in weimar.freifunk.net with OLSR1

@rotanid rotanid added 2. status: rfc request for comments 0. type: enhancement The changeset is an enhancement and removed 2. status: rfc request for comments labels Aug 22, 2016
SvenRoederer pushed a commit to SvenRoederer/freifunk-gluon_core that referenced this issue Sep 29, 2019
Included changes:

a66c088 luci-app-firewall: limit zone name length to 11 characters
4b048cd applications/asterisk: Remove incorrect dependency
eb1ff5b Move libubus-lua dependency to luci-base
d38c239 luci-app-diag-devinfo: mark broken due to dependencies
103e5a3 luci-app-statistics: Adjust ping graphs to show target hosts separately
ae4f8d5 luci-app-statistics: improve scaling of the associated stations graph
18d9c67 luci-app-statistics: backport 'reorder interface and netlink datasources'
8a9ff2b luci-app-statistics: add support for sorting RRD data sources
d4b293b luci-app-statistics: improve diagram generation, add missing title
7b3fea1 luci-app-statistics: rework graph label handling
c8b12e7 Backport luci-base: filter invalid opkg status lines
ce5c787 for-15.05 opkg/packages: Show package size in list of available packages
3e19939 for-15.05: Sync translations
fcc24db luci.mk: correct SK language name to Slovak
ce4ee38 luci-app-statistics: reorganise menu items
321864a luci-base: change index.html to be more like current themes
4bff628 luci-base: Add cache control in index.html
94d8e86 Timezone information: update to 2015g
8832d53 luci-mod-admin-full: status: survive broken DSL status output
f21eb78 luci-app-statistics: only render index view for more than one instance
af9f093 luci-mod-admin-full: fix dnsmasq no-hosts/addn-hosts options
6787a0a for-15.05 luci-base: set default mediaurlbase to bootstrap (default theme)
0b72c51 for-15.05 luci-mod-admin-full: opkg config / prevent word-wrap
5e7c0f0 for-15.05 luci-mod-admin-full: restore opkg feed config capability
720f76c for-15.05 luci-app-firewall: use maxlength datatype for zone name validation
342af52 Merge pull request freifunk-gluon#486 from dwmw2/for-15.05
75327e3 luci/statistics: Fix nut UPS graphs
30f6fe8 Merge pull request freifunk-gluon#474 from hnyman/entropy-1505
d91f0ef statistics: remove references to Lucid from scripts
8e156d6 statistics: adjust default settings to match default plugins
a2a61aa statistics: cleanup config file
8b1de85 for-15.05 statistics: Add support for entropy stats
3836b45 Luci opkg/packages: Limit version string display to 26 chars
b179283 statistics: fix typo
8d2b570 Merge pull request freifunk-gluon#455 from hnyman/backport-stats
7167d97 statistics: clarify CPU/processor graph by removing "idle" from it
27ca079 statistics: clarify stats introduction
f6a4436 statistics: memory plugin - improve graph by better scaling of y-axis
119eaf2 statistics: support rrdtool's alt_autoscale and alt_autoscale_max options
18593ec statistics: cpu graph - add label definitions, add softirq and interrupt stats
791ca8b Delete luci-upnp
21cf10c Merge pull request freifunk-gluon#445 from hnyman/for-15.05
36a7fb4 statistics: fix ping graph label regression
22f687d http.protocol: Support filehandlers for unhandled encodings
7d8163e Merge pull request freifunk-gluon#442 from hnyman/for-15.05
428d181 for-15:05: Timezone information: update to 2015f
c595f30 luci-app-vnstat: Fix blank graphs for iface names with underscores
SvenRoederer pushed a commit to SvenRoederer/freifunk-gluon_core that referenced this issue Sep 29, 2019
7589804 Merge pull request freifunk-gluon#474 from ecsv/batadv-for-18.06
c07326c batman-adv: Fix duplicated OGMs on NETDEV_UP
cad1fba Merge pull request freifunk-gluon#469 from ecsv/batadv-for-18.06
145ba7f batman-adv: Merge bugfixes from 2019.2
40b7519 batman-adv: Reorder patches
e5fe4b6 Merge pull request freifunk-gluon#462 from ecsv/batadv-18.06
ee2d981 batman-adv: Merge bugfixes from 2019.1
4d7a182 nodogsplash: fix invalid pointer bug when clock is turned back (freifunk-gluon#456)
2ad165c Merge pull request freifunk-gluon#452 from dangowrt/openwrt-18.06
71f9aae luci-app-bmx7: update to v0.1-alpha
0e3d701 bmx7: update to git snapshot as of 2018-12-29
42af835 batman-adv: Refresh patches
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
0. type: enhancement The changeset is an enhancement
Projects
None yet
Development

No branches or pull requests

7 participants