tbnobody / OpenDTU

Software for ESP32 to talk to Hoymiles/TSUN/Solenso Inverters
GNU General Public License v2.0
1.78k stars 496 forks source link

no return from manual ip configuration to dhcp #2139

Open grasmax opened 2 months ago

grasmax commented 2 months ago

What happened?

After https://github.com/tbnobody/OpenDTU/issues/2138 - manually ip config .39- I tried to return to automatic ip by checking settings/network/dhcp but opendtu keeps .72 Unchecking dchc, clearing all ip fields and saving, Opendtu says "Error" and restores all IP fields

To Reproduce Bug

check or check not dhcp if the is marked as fix ip in the router

Expected Behavior

if dhcp is ckecked, OpenDtu should have the ip from the router even if all private ip fields are filled or not

Install Method

Pre-Compiled binary from GitHub

What git-hash/version of OpenDTU?

v24.1.4

Relevant log/trace output

No response

Anything else?

No response

Please confirm the following

stefan123t commented 1 month ago

Es sollte normal gar nicht nötig seine irgendeine ip irgendwo zu löschen. Einfach auf dhcp stellen und gut. Dann werden die manuellen ips nie verwendet.

Warum er nicht einfach die alte IP drin lässt und einfach den Schalter wieder auf DHCP stellt weiß ich nicht. Vielleicht will er die Formular-Validierungsfunktion des IP-Adressen Dialogs ausprobieren, die schlägt natürlich an wenn er die statische IP Konfiguration rauslöscht und das so speichern will: IP address is invalid!

Bei mir stand in den Feldern unter Static IP Configuration überall 0.0.0.0 drin, so lässt sich das auch wieder speichern: v24.6.29

Dann hat er trotz statischer IP 0.0.0.0 bei mir wieder die alte DHCP Adresse zugeordnet. Die OpenDTU konnte ich dann unter dem alten Namen http://opendtu-abcdef/ statt der statischen IP (http://192.168.178.7/) wieder aufrufen.

Dann noch den DHCP Haken wieder angemacht und es geht wie von Euch erwartet.

Danke das wars: wenn man den Haken bei DHCP rausnimmt und 0.0.0.0 in alle fünf Felder der "Statischen Konfiguration" einträgt, ist das Speichern erfolgreich. Damit hätte sich zumindest https://github.com/tbnobody/OpenDTU/issues/2139 erledigt.

@tbnobody hier also die Anforderung 0.0.0.0 als Default für das Feld IP Adresse vorzubelegen, falls man es rauslöscht und so speichern will.

@grasmax Ich glaube die beiden Issues #2138 & #2139 können dann als gelöst geschlossen werden ?