MQTT embedded broker has to set its own password#15929
Merged
balloob merged 1 commit intohome-assistant:devfrom Aug 13, 2018
Merged
MQTT embedded broker has to set its own password#15929balloob merged 1 commit intohome-assistant:devfrom
balloob merged 1 commit intohome-assistant:devfrom
Conversation
ghost
assigned
19 tasks
awarecan
changed the title
awarecan
force-pushed
the
mqtt-password
branch
from
e9f3bdd to
49945b5
Compare
Member
|
I didn't see this one yet and created an auth plugin for hbmqtt to make it work with new auth system 👍 will rebase on this PR. |
8 tasks
2 tasks
balloob
pushed a commit
that referenced
this pull request
Aug 14, 2018
2 tasks
Merged
|
Couldn't find how to migrate using https://www.home-assistant.io/docs/mqtt/broker#embedded-broker , found it here. It would be nice to explain it there |
Contributor
|
So how do I migrate? |
Member
|
Don't use old PRs to ask for help/report issues. We have forums and chat for that. From the docs: https://www.home-assistant.io/docs/mqtt/broker#embedded-broker mqtt:
password: YOUR_MQTT_PASSWORD |
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to subscribe to this conversation on GitHub.
Already have an account?
Sign in.
5 participants
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Description:
In order to remove
http.api_passwordin future, we need decouple api_password and other components. MQTT embedded broker has to set its own password.Breaking changes
If user configured
http.api_passwordand used default embedded MQTT broker configuraiton, mqtt component will fail to setup to prompt this change. Error message isRelated issue (if applicable): fixes #
Pull request in home-assistant.github.io with documentation (if applicable): home-assistant/home-assistant.io#5993
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 does not interact with devices: