[VT Server]: Allow responses to messages which do not require a connection with a client#611
Conversation
|
|
FYI I came across an implement which likely failed to load due to this: Unfortunately I have not managed to test with the updated stack as it was at the very end of the Agritechnica :( |
|
1 similar comment
|
|
@ad3154 |
|
I'll keep an eye on the merge into main. Is there a development build available in the meantime? |
|
I'm fine with merging it in, it just doesn't have much testing |
… working set master separately Sometimes a VT client might request certain information from a VT server before it identifies itself as a working set master, and sometimes an external service tool might want to query some details from a VT, so, this allows more messages to be responded to by the VT server without a connection from a client.
ad3154
force-pushed
the
adrian/vt-server-better-connectionless-responses
branch
from
2797b49 to
ed026d3
Compare
|
@martonmiklos I rebased it.. if you approve it we can merge it. |
martonmiklos
deleted the
adrian/vt-server-better-connectionless-responses
branch
3 participants
Describe your changes
Sometimes a VT client might request certain information from a VT server before it identifies itself as a working set master, and sometimes an external service tool might want to query some details from a VT, so, this allows more messages to be responded to by the VT server without a connection from a client.
This will probably improve compatibility with some VT clients.
Specifically, we now allow the following messages from non-clients, or from clients earlier in the connection process.
The diff for this one is going to be large unfortunately... but basically all I did was move the messages into two different functions.
How has this been tested?
I have not yet had the chance to test it in our VT, but intend to test it shortly with #607