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

Jump to bottom

TRACE_MINIMUM log level env val not handled #628

Closed
tucic opened this issue Mar 14, 2019 · 2 comments
Closed

TRACE_MINIMUM log level env val not handled #628

tucic opened this issue Mar 14, 2019 · 2 comments
Milestone
1.3.1

Comments

@tucic
Copy link
Contributor

tucic commented Mar 14, 2019

Condition in Log.c:165 checks for TRACE_MEDIUM, instead of TRACE_MINIMUM. In this way, MQTT_C_CLIENT_TRACE_LEVEL environment variable with value TRACE_MINIMUM will be ignored.
tucic added a commit to tucic/paho.mqtt.c that referenced this issue Mar 14, 2019
@tucic
Fix for TRACE_MINIMUM log level env val not handled (issue eclipse#628)
7ffa643 
Signed-off-by: Milan Tucic <tucic.milan@gmail.com>
@tucic
Copy link
Contributor Author

tucic commented Mar 14, 2019

Fix available in pull request #629.

@icraggs icraggs added this to the 1.3.1 milestone Mar 31, 2020
@icraggs
Copy link
Contributor

icraggs commented Mar 31, 2020

The PR was merged last year. Thanks.

@icraggs icraggs closed this as completed Mar 31, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
1.3.1
Development

No branches or pull requests
2 participants
@icraggs @tucic