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

modbus_server.c: Add print of modbus address #49

Merged
merged 1 commit into from
Dec 16, 2024

Conversation

joakimr-axis
Copy link
Contributor

@joakimr-axis joakimr-axis commented Dec 16, 2024

Describe your changes

Log which incoming address the Modbus request has (but still act on all incoming requests) in server mode. This allows for better testing functionality when using the application in server mode.

Checklist before requesting a review

  • I have performed a self-review of my own code
  • I have verified that the code builds perfectly fine on my local system
  • I have added tests that prove my fix is effective or that my feature works
  • I have commented my code, particularly in hard-to-understand areas
  • I have verified that my code follows the style already available in the repository
  • I have made corresponding changes to the documentation

Log which incoming address the Modbus request has (but still act on all
incoming requests) in server mode. This allows for better testing
functionality when using the application in server mode.

Signed-off-by: Joakim Roubert <[email protected]>
@joakimr-axis joakimr-axis merged commit 4d0be4d into main Dec 16, 2024
8 checks passed
@joakimr-axis joakimr-axis deleted the joakimr-axis_addrserverside branch December 16, 2024 12:15
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Development

Successfully merging this pull request may close these issues.

2 participants