Skip to content
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

Configuration adjustments #228

Open
wants to merge 2 commits into
base: master
Choose a base branch
from
Open

Configuration adjustments #228

wants to merge 2 commits into from

Conversation

raivisdejus
Copy link

The following configuration adjustment will make Mosquitto work by default. It is assumed that users who will set their password will disable anonymous access anyway. This config will work by default with out a password.

Also adding a note about a configuration setting for zigbee2mqtt with Electrolama dongle

@Paraphraser
Copy link
Contributor

Hi,

Please see This Project is Dormant. I haven't checked but you may find that these changes have already been made on the active repo.

Paraphraser added a commit to Paraphraser/IOTstack that referenced this pull request Feb 6, 2021
References:

* [Mosquitto doc](https://mosquitto.org/documentation/migrating-to-2-0/)
* [Workaround](SensorsIot#265)
* [IOTstack new menu breaks Mosquitto configs](SensorsIot#258 (comment))
* [Pull Request on gcgarner/IOTstack](gcgarner#228)

Mosquitto 2.x has introduced a requirement for a `listener` in the
config file. If this is absent, Mosquitto will bind to the loopback
interface. When Mosquitto is running in a container, this implies no
connectivity.

Mosquitto 2.x has also changed the security defaults. Earlier versions
permitted passwordless access out-of-the box. The new version assumes
a password scheme will be implemented and requires passwordless
access to be an explicit configuration choice.

This PR maintains the IOTstack status quo by:

1. Adding `listener 1883`; and
2. Activating `allow_anonymous` with the value `true`.

Anyone who has already set up a password scheme will need to import
the `listener` line but, presumably, their `allow_anonymous` will
already be active and set to `false`, and that will not need to change.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants