search

OpenEVSE / openevse_esp32_firmware

OpenEVSE V4 WiFi gateway using ESP32
154 stars 107 forks source link

Add host name to LCD cycle #145

Open glynhudson opened 3 years ago

glynhudson commented 3 years ago

I think it would be useful to display the IP address and the hostname of the EVSE on the menu cycle, what do you think @chris1howell? I know the IP address and hostname are displayed at statup but if the IP address changes or the user forgets the hostname it can sometimes be tricky to power cycle and get back to the display in time to see the current IP address!

chris1howell commented 3 years ago

I am not sure about this one. Presenting an IP address all the time seems like a security risk.

jeremypoulter commented 3 years ago

Could use the long press of the button to show the IP address/host name like pressing the button on the WiFi module?

glynhudson commented 3 years ago

I don't think it's a big security issue since the perpetrator would need to be connected to your network to be able to access the charge point, once they are connected IP addresses are trivial to scan.

On Wed, 17 Feb 2021 at 22:13, Jeremy Poulter notifications@github.com wrote:

Could use the long press of the button to show the IP address/host name like pressing the button on the WiFi module?

— You are receiving this because you authored the thread. Reply to this email directly, view it on GitHub, or unsubscribe.

-- Glyn Hudson

https://openenergymonitor.org http://zerocarbonadventures.co.uk

slavkobrzeg commented 3 years ago

I had the same problem a couple times when I forgot the IP or hostname and only a hard power reset helped me that time. Although, I'm not a fan of showing it all the time either. It should be accessible but upon the user known action. Maybe that one suggest by @jeremypoulter

glynhudson commented 1 year ago

I think that adding the hostname to the LCD cycle would be a really useful addition, a lot of users struggle to connect to the unit once it's connected to their LAN. I don't think showing the hostname is a security risk e.g openevse-xxxx. This would also make it possible to distinguish between units when several units are installed in the same location.

KipK commented 1 year ago

Perhaps this should be activated by default, and can be disabled from UI config ?