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

Setting "connection_messages false" still logs a "New connection from X on port Y." message #537

Closed
cjmaynard opened this issue Aug 30, 2017 · 2 comments
Labels
Component: mosquitto-broker Type: Enhancement A new feature for a minor or major release.

Comments

@cjmaynard
Copy link

Regarding the connection_messages option, the mosquitto.conf man page reads:

   connection_messages [ true | false ]
       If set to true, the log will include entries when clients connect
       and disconnect. If set to false, these entries will not appear.

With the default of "connection_messages true", and using mosquitto_pub to test, I see messages like so:

1504137323: New connection from 127.0.0.1 on port 1883.
1504137323: New client connected from 127.0.0.1 as mosqpub|2501-test (c1, k60).
1504137323: Client mosqpub|2501-test disconnected.

... but with "connection_messages false", I still see this message:

1504137889: New connection from 127.0.0.1 on port 1883.

I would not expect to see that message when setting "connection_messages false".

@karlp
Copy link
Contributor

karlp commented Aug 30, 2017

the variable name is perhaps bad, in that it says connections, but the manual text says it's the logs on clients, and that's true? I'd probably generally agree though, that it's not the ideal behaviour

@toast-uz
Copy link
Contributor

toast-uz commented Sep 2, 2017

+1
I agree this option is not implemented consistently.

szarta added a commit to szarta/mosquitto that referenced this issue Nov 6, 2017
…n to log it disabled

Signed-off-by: Brandon Arrendondo <[email protected]>
szarta added a commit to szarta/mosquitto that referenced this issue Nov 7, 2017
@toast-uz toast-uz added Type: Enhancement A new feature for a minor or major release. Component: mosquitto-broker labels Mar 2, 2018
ralight added a commit that referenced this issue Dec 9, 2018
Closes #772. Closes #613. Closes #537.

Thanks to Christopher Maynard, Brandon Arrendondo, and qubeck.
@lock lock bot locked as resolved and limited conversation to collaborators Aug 7, 2019
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
Component: mosquitto-broker Type: Enhancement A new feature for a minor or major release.
Projects
None yet
Development

No branches or pull requests

3 participants