Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Expose the device's state (connected or disconnected) #11

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

theckman
Copy link

@theckman theckman commented Nov 23, 2020

If a UniFi device is offline, whether intentionally or accidentally, this
currently breaks the unifi_exporter. By extending this package, we can fix
that bug in unifi_exporter.

@theckman
Copy link
Author

@mdlayher wanted to directly ping, since this will allow me to fix a bug triggering a panic in unifi_exporter. I've a wireless AP that's only connected sometimes (used for meshing), and when it's offline it breaks my stats collection because one of the stats structs is nil.

Would you be willing to have this merged into master? I could probably also take a look at the other PRs and see if they are applicable (if you wanted to give me write access to the repo as well). Alternatively, happy to fork the repo.

If a UniFi device is offline, whether intentionally or accidentally, this
currently breaks the `unifi_exporter`. By extending this package, we can fix
that bug in `unifi_exporter`.
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.

1 participant