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

fix: Change MQTT publish log to debug #22063

Merged
merged 1 commit into from
Apr 3, 2024
Merged

Conversation

Koenkk
Copy link
Owner

@Koenkk Koenkk commented Apr 3, 2024

No description provided.

@Koenkk Koenkk enabled auto-merge (squash) April 3, 2024 18:12
@Koenkk Koenkk merged commit bbb6f9e into dev Apr 3, 2024
15 checks passed
@Koenkk Koenkk deleted the fix/mqtt-publish-log-level branch April 3, 2024 18:13
@mrmaximas
Copy link

mrmaximas commented Apr 3, 2024

No more mqtt messages in the info mode?

@Koenkk
Copy link
Owner Author

Koenkk commented Apr 4, 2024

Indeed

@echaritos
Copy link

Does this mean that we will not be able to log info messages?

@Koenkk
Copy link
Owner Author

Koenkk commented Apr 28, 2024

Does this mean that we will not be able to log info messages?

Info messages are still logged

@martyzeq
Copy link

martyzeq commented May 6, 2024

sorry but this change makes me sad... Is there option to keep previous behaviour somehow ? In my case Info loglevel tells not enough and debug too much - maybe something like syslog loglevel 0-7 would help. Thank you, everything else works great for me.

@jakwarrior
Copy link

I totally agree with that, the previous system was much better to see mqtt messages through the logs page.

@echaritos
Copy link

I find the structure of the log messages now more reliable to extract information. The logs webpage does not work on my system.

@martyzeq
Copy link

martyzeq commented May 6, 2024

I prefer to send logs to remote syslog, but there is a big gap between current Info loglevel and Debug, which I'd like to see filled. Sometimes I just need to backtrace what happened few days ago with some device, and having debug turned on permanently is sledgehammer to crack a nut ...

@Koenkk
Copy link
Owner Author

Koenkk commented May 6, 2024

sorry but this change makes me sad... Is there option to keep previous behaviour somehow ?

Could you create a feature request for this? Then we can keep track how many people are interested in this.

@theDiverDK
Copy link

I support the #22522 improved logging, so we get the info back from before ;-)

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.

6 participants