-
-
Notifications
You must be signed in to change notification settings - Fork 30.4k
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
Add login_method config option to fix login issue with RouterOS Version > 6.43 #25194
Conversation
Is this a breaking change? Does the user need to do anything? |
@MartinHjelmare |
If we can solve it nicely without requiring user interaction, that would be the best. |
@MartinHjelmare Just pushed a commit in which I changed the login method tuple so that users don't have to take action. If they want to avoid the log message on their router, they still can set the login_method in the config and specify which method to use. |
…ge work as expected
What is the log message that can be avoided? |
Its |
Sounds good. Let's make sure we explain clearly in the docs how the user can avoid the log message by configuration. |
…into dev * 'dev' of https://github.com/home-assistant/home-assistant: (156 commits) Add HmIP-PCBS2, HmIP-PCBS-BAT to Homematic IP Cloud (home-assistant#25201) Simplify cache restore (home-assistant#25186) Only include target temp if has right support flag (home-assistant#25193) Fix issue home-assistant#24495 (home-assistant#25199) Handle somfy expired token (home-assistant#25195) Add login_method config option to fix login issue with RouterOS Version > 6.43 (home-assistant#25194) Add HMIP-FCI / HMIP-FBL / HmIP-BBL (home-assistant#25188) [climate] Tweak evohome migration (home-assistant#25187) Fix device name customization on ZHA add devices page (home-assistant#25180) Upgrade mypy to 0.720, turn on unreachability warnings (home-assistant#25157) Use MockConfigEntry (home-assistant#25190) Add severe weather sensor to Dark Sky (home-assistant#22701) Fix typo in azure-pipelines-ci.yml Raise not ready when no data from API is retrieved (home-assistant#25182) Improve Nuki lock (home-assistant#22888) Delete config.yml (home-assistant#25181) Upgrade luftdaten to 0.6.2 (home-assistant#25177) Upgrade Mastodon.py to 1.4.5 (home-assistant#25176) Upgrade ruamel.yaml to 0.15.99 (home-assistant#25175) Upgrade discord.py to 1.2.3 (home-assistant#25174) ...
…into hive_water_heater * 'dev' of https://github.com/home-assistant/home-assistant: (49 commits) Pin Docker to Debain Stretch (home-assistant#25206) Updated frontend to 20190717.0 Add HmIP-PCBS2, HmIP-PCBS-BAT to Homematic IP Cloud (home-assistant#25201) Simplify cache restore (home-assistant#25186) Only include target temp if has right support flag (home-assistant#25193) Fix issue home-assistant#24495 (home-assistant#25199) Handle somfy expired token (home-assistant#25195) Add login_method config option to fix login issue with RouterOS Version > 6.43 (home-assistant#25194) Add HMIP-FCI / HMIP-FBL / HmIP-BBL (home-assistant#25188) [climate] Tweak evohome migration (home-assistant#25187) Fix device name customization on ZHA add devices page (home-assistant#25180) Upgrade mypy to 0.720, turn on unreachability warnings (home-assistant#25157) Use MockConfigEntry (home-assistant#25190) Add severe weather sensor to Dark Sky (home-assistant#22701) Fix typo in azure-pipelines-ci.yml Raise not ready when no data from API is retrieved (home-assistant#25182) Improve Nuki lock (home-assistant#22888) Delete config.yml (home-assistant#25181) Upgrade luftdaten to 0.6.2 (home-assistant#25177) Upgrade Mastodon.py to 1.4.5 (home-assistant#25176) ... # Conflicts: # homeassistant/components/hive/water_heater.py
Description:
Mikrotik changed the used login method with version 6.43 of their RouterOS and disabled the old token login method with version 6.45.1. Therefore the mikrotik component was no longer able to login and use the API.
Related issue: fixes #25189
Pull request with documentation for home-assistant.io : home-assistant/home-assistant.io#9877
Example entry for
configuration.yaml
(if applicable):Checklist:
tox
. Your PR cannot be merged unless tests passIf user exposed functionality or configuration variables are added/changed:
If the code communicates with devices, web services, or third-party tools:
python3 -m script.hassfest
.requirements_all.txt
by runningpython3 -m script.gen_requirements_all
..coveragerc
.If the code does not interact with devices: