MantisBT - Doomseeker
View Issue Details
IDProjectCategoryView StatusDate SubmittedLast Update
0004142Doomseeker[All Projects] Suggestionpublic2023-06-25 13:432024-07-27 15:02
ReporterZalewa 
Assigned ToZalewa 
PrioritynormalSeverityminorReproducibilityN/A
Statusneeds testingResolutionfixed 
PlatformOSOS Version
Product Version 
Target Version1.5.0Fixed in Version1.5.0 
Summary0004142: Support the Zandronum's segmented server query response
DescriptionZandroum is about to gain support for segmented server query response which should help with the response packets going into the jumbo category and getting dropped by the network equipment as a result.

See:'https://wiki.zandronum.com/Launcher_protocol#Segmented_challenge [^]'

Important: The default description points to a backward-incompatible query challenge that changes the packet tag to 200. We don't want that because it will break all previous Zandronum versions and handling this properly would be too convoluted. Keep using the challenge packet with tag 199, but only append the segmentation byte as described further down on the page in the'https://wiki.zandronum.com/Launcher_protocol#Old_protocol_challenge [^]' section.
Steps To Reproduce
Additional InformationSee 0004064 for details.
TagsNo tags attached.
Relationships
related to 0004064resolved DrinkyBird Zandronum Trouble with big UDP packets in launcher query responses from game servers 
parent of 0004147needs testing Zalewa Doomseeker Enable the Zandronum segmented server query by default 
Not all the children of this issue are yet resolved or closed.
Attached Files
Issue History
Date ModifiedUsernameFieldChange
2023-06-25 13:43ZalewaNew Issue
2023-06-25 13:43ZalewaRelationship addedrelated to 0004064
2023-06-25 13:46ZalewaDescription Updatedbug_revision_view_page.php?rev_id=13964#r13964
2023-06-25 13:46ZalewaStatusnew => confirmed
2023-07-22 12:31ZalewaAssigned To => Zalewa
2023-07-22 12:31ZalewaStatusconfirmed => assigned
2023-07-23 18:13ZalewaNote Added: 0022892
2023-07-23 18:13ZalewaStatusassigned => needs review
2023-07-23 18:17ZalewaRelationship addedparent of 0004147
2023-07-23 18:17ZalewaNote Added: 0022893
2023-07-23 22:40ZalewaNote Added: 0022894
2023-07-25 19:29ZalewaNote Added: 0022895
2023-07-25 19:29ZalewaStatusneeds review => needs testing
2023-07-26 21:23ZalewaNote Added: 0022896
2023-07-26 21:23ZalewaStatusneeds testing => resolved
2023-07-26 21:23ZalewaFixed in Version => 1.5.0
2023-07-26 21:23ZalewaResolutionopen => fixed
2024-07-26 14:20ZalewaNote Added: 0023819
2024-07-26 14:20ZalewaStatusresolved => assigned
2024-07-27 14:10ZalewaNote Added: 0023826
2024-07-27 14:10ZalewaStatusassigned => needs review
2024-07-27 15:02ZalewaNote Added: 0023831
2024-07-27 15:02ZalewaStatusneeds review => needs testing

Notes
(0022892)
Zalewa   
2023-07-23 18:13   
Implement the segmented server query in a backwards-compatible way:
-'https://bitbucket.org/Doomseeker/doomseeker/commits/ffda69dc439a4bcb539c0aee120968d75a4e9ded [^]'

And add an opt-in option for it:
-'https://bitbucket.org/Doomseeker/doomseeker/commits/ec453de441bfe38f27c3b0c63506938c45968413 [^]'

I have decided to make this an opt-in only for a short while. The option will stay for now, at most as long as Zandronum itself supports the single packet reply, but it needs to be made enabled by default sooner than that -- in the next release of Doomseeker (or even before that). I will open a separate ticket for making the option enabled by default.
(0022893)
Zalewa   
2023-07-23 18:17   
Option default -> 0004147.
(0022894)
Zalewa   
2023-07-23 22:40   
Fixes:
1.'https://bitbucket.org/Doomseeker/doomseeker/commits/ae38a9552198243fd9b6e66cccba0bff59fcb882 [^]'
2.'https://bitbucket.org/Doomseeker/doomseeker/commits/c39aa329203fa6953d7e2acf65875f25d11134a0 [^]'
3.'https://bitbucket.org/Doomseeker/doomseeker/commits/5ee17c7d4252a9f0cc400b757c96cccaac976efc [^]'
(0022895)
Zalewa   
2023-07-25 19:29   
Ready for testing:'https://devbuilds.drdteam.org/doomseeker/doomseeker-1.4.1~beta-230723-2106_windows.zip [^]'
(0022896)
Zalewa   
2023-07-26 21:23   
This issue is confirmed by Iko and Kaminsky on Discord as fixed.
(0023819)
Zalewa   
2024-07-26 14:20   
The protocol was altered on 2024-07-21 to "segmented reply v2" and Doomseeker needs to be adapted.

'https://wiki.zandronum.com/w/index.php?title=Launcher_protocol&oldid=2119 [^]'
(0023826)
Zalewa   
2024-07-27 14:10   
Segmented reply v2:
'https://bitbucket.org/Doomseeker/doomseeker/commits/7545e7cdd46c3b9bfaabe7c3d1119f045c9dde4d [^]'
(0023831)
Zalewa   
2024-07-27 15:02   
Segmented reply v2 is available for testing in this beta build:'https://devbuilds.drdteam.org/doomseeker/doomseeker-1.4.1~beta-240727-1427_windows.zip [^]'

There's currently no Zandronum testing build that supports this, though. At the moment the latest ZandroDev is 3.2-240414-1910, while the segmented reply v2 was authored on 2024-05-19.