Skip to content

Embedded MQTT broker needs setup its own password#5993

Merged
frenck merged 3 commits intonextfrom
awarecan-patch-1
Aug 13, 2018
Merged

Embedded MQTT broker needs setup its own password#5993
frenck merged 3 commits intonextfrom
awarecan-patch-1

Conversation

@awarecan
Copy link
Copy Markdown
Contributor

@awarecan awarecan commented Aug 11, 2018

Description:

Pull request in home-assistant (if applicable): home-assistant/core#15929

Checklist:

  • Branch: Fixes, changes and adjustments should be created against current. New documentation for platforms/components and features should go to next.
  • The documentation follow the standards.

@ghost ghost assigned awarecan Aug 11, 2018
@ghost ghost added the ready-for-review This PR needs to be reviewed label Aug 11, 2018
Comment thread source/_docs/mqtt/broker.markdown Outdated
| Password | no default value |
| Websocket port | 8080 |

> Since 0.77 release, embedded broker doesn't use your [API password](/components/http/) as default value. MQTT component will fail to setup if you have API passowrd configured, but don't have MQTT password confgiured likes following example.
Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Couple of typo/spelling errors, and maybe add some "the"s for better reading flow.

Since 0.77 release, the embedded broker doesn't use your API password as default password. The MQTT component will fail to setup if you have API password configured, but don't have MQTT password configured as in the following example.

@frenck frenck added next This PR goes into the next branch has-parent This PR has a parent PR in another repo new-feature This PR adds documentation for a new Home Assistant feature to an existing integration labels Aug 13, 2018
@frenck frenck self-requested a review August 13, 2018 10:34
@frenck frenck self-assigned this Aug 13, 2018
@frenck
Copy link
Copy Markdown
Member

frenck commented Aug 13, 2018

Did a minor tweak.
✅ Approved. Can be merged as soon as the build passes.

@frenck frenck added awaits-parent Awaits the merge of an parent PR and removed ready-for-review This PR needs to be reviewed labels Aug 13, 2018
@frenck frenck merged commit 8ff94bf into next Aug 13, 2018
@frenck frenck deleted the awarecan-patch-1 branch August 13, 2018 18:25
flo-wer pushed a commit to flo-wer/home-assistant.io that referenced this pull request Aug 17, 2018
* Embedded MQTT broker needs setup its own password

* Update broker.markdown

* ✏️ Minor tweak
flo-wer pushed a commit to flo-wer/home-assistant.io that referenced this pull request Aug 17, 2018
* Embedded MQTT broker needs setup its own password

* Update broker.markdown

* ✏️ Minor tweak
@c0ffeeca7 c0ffeeca7 removed the awaits-parent Awaits the merge of an parent PR label Sep 16, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

has-parent This PR has a parent PR in another repo new-feature This PR adds documentation for a new Home Assistant feature to an existing integration next This PR goes into the next branch

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants