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

Netzwerkeinstellungen - Felder für die "fixen Netzwerkeinstellungen" mit sinnvollen Daten vorbelegen #2394

Closed
Baxxy13 opened this issue Aug 21, 2023 · 2 comments · Fixed by #2397
Labels
💡 enhancement-ideas New feature or change request

Comments

@Baxxy13
Copy link
Contributor

Baxxy13 commented Aug 21, 2023

Describe the solution you'd like

Der allererste Start von RM erfolgt mittels Netzwerkkonfiguration über DHCP.
Will man dann (später) auf fixe Netzwerkeinstellungen umstellen muss man den Haken setzen und die Template-Daten selbstständig durch plausible Daten ersetzen.

Für "Netzwerker" kein Problem, aber der ein oder andere Laie kommt da vielleicht ins straucheln.

Die Idee ist also die "Template-Daten" sinnvoll vorzubelegen, vielleicht einfach mit den Daten die per DHCP ermittelt wurden.

Describe alternatives you've considered

Is your feature request related to a problem?

Eher ein Komfortgewinn statt wirkliches Problem.
Das würde die Umstellung von DHCP auf "fixe Konfig" erleichtern.
Auch wenn bisher scheinbar nur einer von 100 die Umstellung versemmelt hat. 😉

Diskussion im Homematic-Form

Additional information

Wie man das am besten umsetzt weiß ich aktuell nicht.

Meine Ideen dazu wären aber:

  • Beim RM-Erststart einmalig die Templates mit den per DHCP ermittelten Werten ausfüllen
  • Einen Button nutzen der die DHCP-Konfiguration für die Templates übernimmt.
@Baxxy13 Baxxy13 added the 💡 enhancement-ideas New feature or change request label Aug 21, 2023
@jp112sdl
Copy link
Contributor

jp112sdl commented Aug 21, 2023

Ich würde die Zeilen der /www/config/cp_network.cgi#L908-L912 :

  if {! [get_property $netconfig "IP" ip] } {return 0}
  if {! [get_property $netconfig "NETMASK" mask] } {return 0}
  if {! [get_property $netconfig "GATEWAY" gw] } {return 0}
  get_property $netconfig "NAMESERVER1" dns1
  get_property $netconfig "NAMESERVER2" dns2

ändern in

  set current_prefix ""
  if { $dhcp } { set current_prefix "CURRENT_" }
  if {! [get_property $netconfig "${current_prefix}IP" ip] } {return 0}
  if {! [get_property $netconfig "${current_prefix}NETMASK" mask] } {return 0}
  if {! [get_property $netconfig "${current_prefix}GATEWAY" gw] } {return 0}
  get_property $netconfig "${current_prefix}NAMESERVER1" dns1
  get_property $netconfig "${current_prefix}NAMESERVER2" dns2

Wenn DHCP aktiviert ist, werden in den Edit-Feldern die aktuellen Werte (beginnend mit CURRENT_ aus der netconfig-Datei) eingetragen.

Wird DHCP dann deaktiviert, bleiben die Werte auch so erhalten, so dass der Anwender im besten Fall nur das letzte Oktett ändert.

Ist der manuelle Modus aktiv, dann bleibt das Auslesen der netconfig wie bisher.

Diese aus meiner Sicht recht einfache Umsetzung mit wenig Umbauten würde jedoch momentan mit dem Patch 0136-WebUI-Add-Tailscale-VPN kollidieren, da die Anpassungen diesen Block beträfen:

@@ -872,10 +910,14 @@
if {! [get_property $netconfig "GATEWAY" gw] } {return 0}
get_property $netconfig "NAMESERVER1" dns1
get_property $netconfig "NAMESERVER2" dns2
+
+ # check for vpn enable status
+ set vpn [file exists "/etc/config/tailscaleEnabled"]
+
return 1
}

Daher die Frage: Soll ich als .orig die bereits durch dich um tailscale erweiterte cp_network.cgi aus dem Patch 0136-WebUI-Add-Tailscale-VPN nehmen?

@jens-maus
Copy link
Owner

Ja, wenn es da Kollisionen zwischen Patches gibt ist das oftmals die einzige lösung. Müsstest du also bitte so umsetzen in deinem neuen webui patch.

jp112sdl added a commit to jp112sdl/RaspberryMatic that referenced this issue Aug 21, 2023
@jens-maus jens-maus linked a pull request Aug 21, 2023 that will close this issue
7 tasks
jens-maus pushed a commit that referenced this issue Aug 21, 2023
…atch which will display the current ip settings in the network config page if DHCP is enabled (#2397, #2394, @jp112sdl)
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
💡 enhancement-ideas New feature or change request
Projects
None yet
Development

Successfully merging a pull request may close this issue.

3 participants