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

Send multiple message over same TCP connection #32

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

ifan-t
Copy link

@ifan-t ifan-t commented Sep 6, 2023

I have added the ability to send multiple HL7 messages over a persistent TCP connection, which is closed when the ack for the last message sent is received. This is found in the 'SendMultipleMessages' command.
The messages are rendered onto the 'SendHl7MessageWebView' as a single string, and then the same logic used to extract multiple messages in the 'SplitBatchFile' command is used to extract the separate messages from this string into a string array. Each subsequent message is sent over the same TCP connection once an ack is received for the previously sent message, and we use an increment to keep track of the message being processed. Once we reach the end of the array and an ack is received for the last message in the array sent, we close the connection.

* Send multiple message over same TCP connection

* Refactor, get rid of uncessary for loop

* Remove unused code
@RobHolme
Copy link
Owner

RobHolme commented Sep 9, 2023

Thanks, I appreciate the contribution. I'll get onto merging this when I have some free time.

@Aravindhan-314e
Copy link

Is this feature is available now ?

@RobHolme
Copy link
Owner

Hi. When I tested the PR it didn't work for me. I started looking into a different approach that should have been more efficient but ran out of free time to work on this. My weekends are fully committed with household projects at the moment, and likely to be that way for the next 3 months or so. I'll take a look whenever I have free time to dedicate to this, but at this stage it might take a while.

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.

3 participants