Skip to content

Conversation

@x2on
Copy link

@x2on x2on commented Nov 30, 2025

Add the WAN IP address to the details.

Tested with the LM1200.

I would like to use this in the Netgear LTE Home Assistant Addon. So i need this in the Information class.

Add the WAN IP address to the details
@amelchio
Copy link
Owner

Thank you for the contribution. However, I stopped adding fields to the Information class so please just find the data you need in the items field.

The status.py example shows how to extract an arbitrary value:

examples/status.py 192.168.5.1 <password> wwan.ip

@x2on
Copy link
Author

x2on commented Nov 30, 2025

Thank you for the clarification.
I understand that you prefer not to add new fields to the Information class.

In the Home Assistant Core Netgear LTE integration I would need the WAN IP in addition to the existing data. Calling information() a second time only for this value would not be ideal.

Would you be open to introducing a lightweight mechanism to request optional information in a single call, so consumers can retrieve additional values like wwan.ip without extra requests?

Thanks again for your work on this library.

@amelchio
Copy link
Owner

You do not need a second request, all the items are already returned along with the other fields.

This was in fact built for the Home Assistant integration because adding fields one by one got old after a while. However, I never got around to actually add arbitrary sensors to HA.

@x2on
Copy link
Author

x2on commented Nov 30, 2025

Oh I missed that. Thanks for your help. I will have a look at this.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants