Skip to content

Conversation

@Bertimus-maximus
Copy link

This PR will allow users of the library to add their own request handlers for Modbus servers. I have successfully used it to implement a data packet logging scheme.

@Bertimus-maximus
Copy link
Author

Hello, I have also added a request handler for RTU over TCP connections to this PR. There is a test showing its compatibility with the existing RTU Over TCP client. Thanks for all of your work on this excellent library.

@Bertimus-maximus
Copy link
Author

@Apollo3zehn I was wondering if you had gotten a chance to look at this and if you thought there was a path to merge it at some point? Thanks again.

@Apollo3zehn
Copy link
Owner

@Bertimus-maximus Thanks for your pull request. I am having trouble to find the necessary time for this project. For a very long time now I wanted to prepare a v6 with some API changes but I was not yet able to do so. Last week our daughter was born and so I am again a little bit distracted. But I have holidays for 4 weeks now so maybe I will find the time for some FluentModbus improvements and this pull request.

@Bertimus-maximus
Copy link
Author

Congratulations on the birth of your daughter! That's terrific and very much more important than my pull request :)

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.

2 participants