search

tomaae / homeassistant-mikrotik_router

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

HA 2022.10.0 - user does not have sufficient access rights #250

Closed SAOPP closed 1 year ago

SAOPP commented 1 year ago

Hi there!

After upgrading of HA to 2022.10.0 got this warning:

Mikrotik user homeassistant does not have sufficient access rights. Integration functionality will be limited.

...but, I dind't changed anything and have all needed permissions for user from day one.

Here is user and groups of routeros:

image image

Why this warning is pop up?

tomaae commented 1 year ago

this is not related to HA upgrade, but integration update. there is additional functionality that requires additional access rights. you dont have to grant it it you dont want, its up to you. in your case, I think you just wont see firmware and os update sensors. one more will be missing right now, but thats a bug which will be fixed in next release.

see doc for permission info https://github.com/tomaae/homeassistant-mikrotik_router#setup-integration

SAOPP commented 1 year ago

I did't say that this was due to the HA update, I only emphasized that there was no such warning before this update. Regarding the sensors, yes you are right, these sensors have become unavailable now.

Why should I look in the permissions documentation? Has anything changed there lately? I showed the screenshots - they look identical with the necessary.

tomaae commented 1 year ago

Yes, I seen your screenshot. Thats why I told you to look into docs for up to date info. docs have been updated when new sensors and functionality was added.

SAOPP commented 1 year ago

Well, before creating a issue, I checked if there were any discrepancies with the recommended rights for the user, and making sure that everything was up to date with me, I decided to ask the question anyway. In general, we are waiting for updates?

tomaae commented 1 year ago

Well, you are missing "reboot" rights for full functionality. If you mean health sensors bug, then yes, that will be in next update. But I dont plan to work on it anytime soon as its a minor thing and it works fine with full permissions.

SAOPP commented 1 year ago

God damn it, I swear I didn't see it, damn, oh maaaan :disappointed: Stupid things. Sorry for the issue, can't believe, why I skipped reboot permission, and in during my comparison actually. Weird sh...t. Sorry once again buddy, maybe hard day I don't know...

tomaae commented 1 year ago

dont worry about it, such things happen. and after all, no harm done ;)