MayaTheShy/Starworld
1
0
Fork 0
Code Issues Pull Requests 1 Actions Packages Projects Releases Wiki Activity

fix: avoid setting source ID in DomainConnectRequest before local ID is assigned
Some checks failed
CI / build-and-test (push) Has been cancelled
Details

Browse Source
This commit is contained in:
MayaTheShy
2025-11-09 23:58:59 -05:00
parent fe81c69e1a
commit d507902a73
1 changed files with 2 additions and 0 deletions
Download Patch File Download Diff File Expand all files Collapse all files

2
src/OverteClient.cpp
View File

@@ -1235,6 +1235,8 @@ void OverteClient::sendDomainConnectRequest() {
// Create NLPacket with DomainConnectRequest type and correct version // Create NLPacket with DomainConnectRequest type and correct version
NLPacket packet(PacketType::DomainConnectRequest, PacketVersions::DomainConnectRequest_SocketTypes, true); NLPacket packet(PacketType::DomainConnectRequest, PacketVersions::DomainConnectRequest_SocketTypes, true);
packet.setSequenceNumber(m_sequenceNumber++); packet.setSequenceNumber(m_sequenceNumber++);
// DomainConnectRequest is sent BEFORE we have a local ID, so don't set source ID
// The server will assign us an ID in the DomainList response
// Build payload using Qt wire format (match Overte's NodeList.cpp structure exactly) // Build payload using Qt wire format (match Overte's NodeList.cpp structure exactly)
QtStream qs; QtStream qs;