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

fix: update NETWORK_PROTOCOL_INVESTIGATION to reflect HMAC verification deadlock and connection persistence issues

Browse Source
This commit is contained in:
MayaTheShy
2025-11-10 01:49:50 -05:00
parent 88de6f5e28
commit 8e2a8a913d
1 changed files with 12 additions and 7 deletions
Download Patch File Download Diff File Expand all files Collapse all files

19
docs/NETWORK_PROTOCOL_INVESTIGATION.md
View File

@@ -2,22 +2,27 @@
## Current Status (Nov 10, 2025) ## Current Status (Nov 10, 2025)
**✅ CONNECTION PERSISTENCE ISSUE RESOLVED!** **⚠️ HMAC VERIFICATION DEADLOCK - SERVER CONFIGURATION ISSUE**
### Working ✅ ### Working ✅
- **DomainConnectRequest packet format** - Matches official Overte client implementation - **DomainConnectRequest packet format** - Matches official Overte client implementation
- **Connection established** - Server responds with DomainList packet - **Connection established** - Server responds with DomainList packet
- **Local ID assignment** - Server assigns a valid Local ID - **Local ID assignment** - Server assigns a valid Local ID correctly
- **Local ID parsing** - Fixed byte order bug (was using ntohs() on little-endian data) - **Local ID parsing** - Fixed to read little-endian uint16 at correct offset (bytes 34-35)
- **Symmetric socket creation** - Server creates correct socket to our actual address - **Symmetric socket creation** - Server creates correct socket to our actual address
- **Packet version negotiation** - Using correct version 27 (DomainConnectRequest_SocketTypes) - **Packet version negotiation** - Using correct version 27 (DomainConnectRequest_SocketTypes)
- **Protocol signature** - MD5 hash `eb1600e798dc5e03c755a968dc16b7fc` matches server - **Protocol signature** - MD5 hash `eb1600e798dc5e03c755a968dc16b7fc` matches server
- **Packet structure** - All fields properly ordered and serialized per Qt QDataStream format - **Packet structure** - All fields properly ordered and serialized per Qt QDataStream format
- **Source ID handling** - Little-endian uint16, correctly set on sourced packets - **Source ID handling** - Little-endian uint16 at offset 6-7 in sourced packet header
- **Connection persistence** - Connection now stays alive indefinitely with correct Local ID ✅ - **HMAC-MD5 implementation** - Correctly calculates hash using OpenSSL with null UUID as key
- **Activity tracking** - Server recognizes our sourced packets (Ping, AvatarData) and updates lastHeardMicrostamp ✅ - **Packet hash insertion** - Properly reserves 16-byte slot and moves payload
### Root Cause of Connection Persistence Bug (FIXED) ### Not Working / Blocked ❌
- **Connection persistence** - Still killed after 11-18 seconds as "silent node"
- **HMAC verification** - Server rejects all sourced packets due to hash mismatch
- **Keep-alive mechanism** - Cannot send valid Ping packets that server will accept
### Root Cause: HMAC Verification Deadlock (UNSOLVED)
The connection was being killed after 16 seconds because the server couldn't match our sourced packets to our node. The connection was being killed after 16 seconds because the server couldn't match our sourced packets to our node.