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.

Sign up for GitHub

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

Add errorcodes to mosquitto_strerror. Closes #2799 #2800

Closed
wants to merge 1 commit into from
Closed

Add errorcodes to mosquitto_strerror. Closes #2799 #2800

wants to merge 1 commit into from

Conversation

n00b42
Copy link

@n00b42 n00b42 commented May 3, 2023

Add string representations for MOSQ_ERR_KEEPALIVE and MOSQ_ERR_LOOKUP. Closes #2799

  • Have you signed the Eclipse Contributor Agreement, using the same email address as you used in your commits?
  • Do each of your commits have a "Signed-off-by" line, with the correct email address? Use "git commit -s" to generate this line for you.
  • If you are contributing a new feature, is your work based off the develop branch?
  • If you are contributing a bugfix, is your work based off the fixes branch?
  • Have you added an explanation of what your changes do and why you'd like us to include them?
  • Have you successfully run make test with your changes locally?

@n00b42
Add errorcodes to mosquitto_strerror. Closes eclipse#2799
f4f050a 
Add string representations for MOSQ_ERR_KEEPALIVE and MOSQ_ERR_LOOKUP.
Closes eclipse#2799
@ralight
Copy link
Contributor

ralight commented Aug 11, 2023

Thank you. This is already fixed in the develop branch, but it makes sense to port it back. I prefer my entry for the keepalive message, but have used the clearer version of the lookup error that you have provided.

@ralight ralight closed this Aug 11, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

mosquitto_strerror does not cover MOSQ_ERR_KEEPALIVE and MOSQ_ERR_LOOKUP
2 participants
@n00b42 @ralight