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

New connection per message handler request #5

Open
deavmi opened this issue Apr 25, 2020 · 3 comments
Open

New connection per message handler request #5

deavmi opened this issue Apr 25, 2020 · 3 comments
Assignees
@deavmi
Labels
enhancement New feature or request
Milestone
Full implementation

Comments

@deavmi
Copy link

deavmi commented Apr 25, 2020

Currently the handler is connected to on startup. We should rather make several connections ad open and close the socket when doing message handling.
@deavmi deavmi added the enhancement New feature or request label Apr 25, 2020
@deavmi deavmi added this to the Full implementation milestone Apr 25, 2020
@deavmi deavmi self-assigned this Apr 25, 2020
@deavmi
Copy link
Author

deavmi commented Apr 25, 2020

Added function for spawning a new socket: 60d0d09

Still to do, use getNewSocket and then also remove old code connecting on startup.

@deavmi
Copy link
Author

deavmi commented Apr 26, 2020

Done as of: 0ac8e56

@deavmi
Copy link
Author

deavmi commented Apr 26, 2020

I must just clean up the left over code now.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@deavmi deavmi

Labels
enhancement New feature or request
Projects
None yet
Milestone
Full implementation
Development

No branches or pull requests
1 participant
@deavmi