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

Mosquitto changes #4

Merged
merged 3 commits into from
Apr 20, 2020
Merged

Mosquitto changes #4

merged 3 commits into from
Apr 20, 2020

Conversation

Slyke
Copy link
Collaborator

@Slyke Slyke commented Apr 19, 2020

Hi. I installed IOTstack recently and ended up making this changes. They are basic and can be useful for security.

This is from: gcgarner#130 Contributor: @Tomasmerca

Route filter.acl to add the option in mosquitto/mosquitto.conf of filtering wich topics can be read or writen by defined users.
Added the file where the user creates filters.
The example filters don´t affect usage in any way.
@Slyke Slyke changed the title Pr/130 Mosquitto changes Apr 19, 2020
@Slyke
Copy link
Collaborator Author

Slyke commented Apr 20, 2020

Have tested the change with and without authentication username and password set (to localhost). Works fine.

@Willem-Dekker
Copy link
Collaborator

did you test it with them set?

@Slyke
Copy link
Collaborator Author

Slyke commented Apr 20, 2020

Yup! Tested with and without. Here's a screenshot with it set:
image

@Willem-Dekker Willem-Dekker merged commit d86a5cd into master Apr 20, 2020
@Willem-Dekker Willem-Dekker deleted the pr/130 branch April 20, 2020 08:11
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.

3 participants