search

tomaae / homeassistant-mikrotik_router

Mikrotik router integration for Home Assistant
Apache License 2.0
301 stars 50 forks source link

[Bug] missing 'test' permission from Setup Integration section #244

Closed mvdwetering closed 2 years ago

mvdwetering commented 2 years ago

Describe the issue

I noticed in the logs it mentioned that the integration was running in limited mode because the user on the router did not have enough permissions.

To fix it I looked at the Setup Integration section and updated the permissions to "read, write, api, reboot, policy" as described. After rebooting HA I got the following lines in the logs.

Mikrotik 192.168.178.9 error while arp_ping : not enough permissions (9)

I remembered that I removed the 'test' permission that I had enabled before, so I put that back and after a HA reboot the messages went away.

How to reproduce the issue

Update HA user permissions on the router to not have 'test'

Expected behavior

Documentation covers all permissions

Screenshots

n.a.

Software versions

Diagnostics data

Traceback/Error logs

This error originated from a custom integration.

Logger: custom_components.mikrotik_router.mikrotikapi Source: custom_components/mikrotik_router/mikrotikapi.py:375 Integration: Mikrotik Router (documentation, issues) First occurred: 20:53:02 (19 occurrences) Last logged: 20:53:02

Mikrotik 192.168.178.9 error while arp_ping : not enough permissions (9)

Additional context

tomaae commented 2 years ago

thanks, I removed if because I could not figure out what it was for. no idea how I missed arp ping