Hi,
Gesendet: Samstag, 14. September 2019 um 15:55 Uhr Von: "Matthias Fischer" matthias.fischer@ipfire.org An: development@lists.ipfire.org Betreff: Re: Confusing error message
Hi,
On 14.09.2019 15:29, Tapani Tarvainen wrote:
I just spent a stupid amount of time trying to figure out why IpFire insisted a fixed lease had "invalid fixed ip address" when I could see nothing wrong with the address.
I had to read the source before I realized where the problem was: the same error message is also used when the *next-server* address is bad, and it doesn't accept server name there but wants an IP.
To be sure: do you mean this if-clause in 'dhcp.cgi'?
... if ($dhcpsettings{'FIX_NEXTADDR'}) { unless(&General::validip($dhcpsettings{'FIX_NEXTADDR'})) { $errormessage = $Lang::tr{'invalid fixed ip address'}; } } ...
I guess this would not be the most critical thing to fix but at least changing the error message would be nice.
Yep. Should be easy.
Adjusting 'dhcp.cgi' and the 'lang'-string for the errormessage should be sufficient. I just need your confirmation.
Would the errormessage 'invalid ip' => 'Invalid IP Address' be clear enough/sufficient?
Alternative: 'invalid next-server ip' => 'Invalid IP Address for next-server'
I would prefer this version. The semantic "next-server must be a IP address" is expressed more clearly.
Best, Matthias
Best, Bernhard