treewide: Split home.html in ssl-webui #94

ChristianD · 2021-01-29T08:55:15+01:00

ChristianD commented

2021-01-29 08:55:15 +01:00

We have different VPN or other things in the home.html of the ssl-webui

The patch split the files to the correct packages.

The layer3 file is a little bit smaller and need some work in feature to
add some informations (e.g. wireguard or babel information)

Signed-off-by: Christian Dresel freifunk@dresel.systems

We have different VPN or other things in the home.html of the ssl-webui The patch split the files to the correct packages. The layer3 file is a little bit smaller and need some work in feature to add some informations (e.g. wireguard or babel information) Signed-off-by: Christian Dresel <freifunk@dresel.systems>

ChristianD added 1 commit 2021-01-29 08:55:15 +01:00

1e373481cf treewide: Split home.html in ssl-webui

We have different VPN or other things in the home.html of the ssl-webui

The patch split the files to the correct packages.

The layer3 file is a little bit smaller and need some work in feature to
add some informations (e.g. wireguard or babel information)

Signed-off-by: Christian Dresel <freifunk@dresel.systems>

adschm commented

2021-01-29 12:08:40 +01:00

Ich bin nach wie vor sehr überzeugt davon, dass Sachen aus den web*-Packages in Nicht-web-Packages nichts zu suchen haben.

ChristianD commented

2021-01-29 13:00:12 +01:00

D.h. du hättest hier gerne 2 neue Packages:

fff-web-layer3home
fff-web-nodehome

?

D.h. du hättest hier gerne 2 neue Packages: fff-web-layer3home fff-web-nodehome ?

adschm commented

2021-01-29 14:01:35 +01:00

Ich hätte gerne, dass man das in einer vorhandenen Package irgendwie schaltet (habs mir noch nicht im Detail angekuckt).

In jedem Fall ist mir aber grundsätzlich eben wichtig, dass man das komplette WebUI abwählen kann, wenn man es nicht will, ohne dass dann was kaputt geht oder Leichen in anderen Packages bleiben.

Ich hätte gerne, dass man das in einer vorhandenen Package irgendwie schaltet (habs mir noch nicht im Detail angekuckt). In jedem Fall ist mir aber grundsätzlich eben wichtig, dass man das komplette WebUI abwählen kann, wenn man es nicht will, ohne dass dann was kaputt geht oder Leichen in anderen Packages bleiben.

ChristianD referenced this pull request

2021-02-11 15:48:25 +01:00

layer3: Bandbreitenbegrenzung nicht möglich. Ergänzen oder aus WebUI entfernen #110

fbl commented

2021-03-02 01:26:42 +01:00

Wenn wir das ganze so wie hier splitten, dann haben wir ziemlich viel doppelten Code. Besser wäre es, nur die entsprechenden Teile, die sich unterscheden, zu ändern. Ich weiß nicht, ob es hier so etwas wie ein include gibt. Ansonsten sollte zur Laufzeit unterschieden werden.

ChristianD commented

2021-03-03 18:24:00 +01:00

Include muss ich mir angucken, bin ich mir jetzt auch nicht sicher.

Zur Laufzeit unterscheiden in der html File? Finde ich ehrlich gesagt schrecklich und will ich eigentlich nicht haben. Dummerweise hab ich aktuell auch keine bessere Idee.

Der Code ist ja nicht doppelt am Router sondern nur hier im Git. Und Node sowie Layer 3 wird sich z.b. an dieser Stelle immer wieder deutlich unterscheiden so das ich schon der Meinung bin, man kann diese Seite getrennt pflegen. Die Leute denen die Seite am Layer 3 wichtig ist, werden das Layer 3 Paket pflegen, die Personen denen die Node Variante wichtig ist, werden die Node Seite pflegen. Das wird sich dann vermutlich irgendwann mehr und mehr unterscheiden (oder auch nicht, wenn die eine Seite irgendwas von der anderen übernimmt) aber macht ja nichts, es sind ja eh grundsätzlich total verschiedene Infos die man hier haben will.

Falls include geht, könnte ich mich damit vllt. noch anfreunden aber irgendwelche

if layer3 then "mach dies"
elseif node then "mach das"

will ich eigentlich nicht da drinnen haben.

Was ich mir vllt. auch noch vorstellen könnte:

if fastd_vorhanden then "zeige fastd infos"
if wireguard_vorhanden then "zeige wg infos"
if babeld_vorhanden then "zeige babeld Infos"
if batman-adv_vorhanden then "zeige batman-adv Infos"
usw.

Also das ganze Paketabhängig zu betrachten, darüber könnte man mal nachdenken. Dann müsste man dies aber auch wieder in diese Packages packen und am Ende braucht man doch wieder includes und dann landen wir wieder bei @adschm seinen Package Problem...

Include muss ich mir angucken, bin ich mir jetzt auch nicht sicher. Zur Laufzeit unterscheiden in der html File? Finde ich ehrlich gesagt schrecklich und will ich eigentlich nicht haben. Dummerweise hab ich aktuell auch keine bessere Idee. Der Code ist ja nicht doppelt am Router sondern nur hier im Git. Und Node sowie Layer 3 wird sich z.b. an dieser Stelle immer wieder deutlich unterscheiden so das ich schon der Meinung bin, man kann diese Seite getrennt pflegen. Die Leute denen die Seite am Layer 3 wichtig ist, werden das Layer 3 Paket pflegen, die Personen denen die Node Variante wichtig ist, werden die Node Seite pflegen. Das wird sich dann vermutlich irgendwann mehr und mehr unterscheiden (oder auch nicht, wenn die eine Seite irgendwas von der anderen übernimmt) aber macht ja nichts, es sind ja eh grundsätzlich total verschiedene Infos die man hier haben will. Falls include geht, könnte ich mich damit vllt. noch anfreunden aber irgendwelche if layer3 then "mach dies" elseif node then "mach das" will ich eigentlich nicht da drinnen haben. Was ich mir vllt. auch noch vorstellen könnte: if fastd_vorhanden then "zeige fastd infos" if wireguard_vorhanden then "zeige wg infos" if babeld_vorhanden then "zeige babeld Infos" if batman-adv_vorhanden then "zeige batman-adv Infos" usw. Also das ganze Paketabhängig zu betrachten, darüber könnte man mal nachdenken. Dann müsste man dies aber auch wieder in diese Packages packen und am Ende braucht man doch wieder includes und dann landen wir wieder bei @adschm seinen Package Problem...

fbl commented

2021-12-30 17:24:46 +01:00

In der aktuellen Form nicht sinnvoll (siehe die beiden Anmerkungen), daher erstmal rejected.

fbl closed this pull request

2021-12-30 17:24:46 +01:00

fbl added the

rejected

label 2021-12-30 17:24:52 +01:00