ckarrie / ha-netgear-plus

HomeAssistant Netgear Switch Integration
Apache License 2.0
21 stars 10 forks source link

Side effects when switching POE port power on GS308EP #27

Open foxey opened 1 month ago

foxey commented 1 month ago

There are some unwanted side effects caused by enabling/disabling the POE Port Power using this integration:

Initial settings:

| Port Priority         | Low             |
| Power Limit Type      | User            |
| Power Limit (W)       | 30.0            |
| Detection Type        | IEEE 802        |
| Longer Detection Time | Disable         |

Settings after switching off and on again:

| Port Priority         | Low             |
| Power Limit Type      | None            |
| Power Limit (W)       | 16.2            |
| Detection Type        |                 |
| Longer Detection Time | Get Value Fault |

The switch probably expects a full set of data in the posted form and overwrites the missing attributes with default/empty values. A way to solve this is to fetch the form first, copy the current settings and only change the port power value before posting back the entire form.

ckarrie commented 1 month ago

Thought about that too, thanks for looking at it!

vanrace commented 3 weeks ago

I can confirm this happens with the GS308EPP as well. Happy to support testing