MantisBT - Doomseeker
View Issue Details
0001711Doomseeker[All Projects] Bugpublic2014-02-20 13:242018-09-29 14:46
Absolute Zero 
Zalewa 
normalblockalways
closedfixed 
MicrosoftWindowsXP/Vista/7
0.11.1 Beta 
0.12 Beta0.12 Beta 
0001711: Get Servers button is grayed out
When an user is updating server list and "Error: Master server for Zandronum: Connection timeout (66.228.33.44:15300)" message occurs, Get Servers button is grayed out.

Log:'http://pastebin.com/pxLxD42k [^]'
Image:'http://i1363.photobucket.com/albums/r704/Zero_Kelvin/dsbug_zpsced8a8eb.jpg [^]'
Only way to reproduce is simulating a connection timeout.
No tags attached.
? Zandronum (10) 2014-03-07 16:18
https://zandronum.com/tracker/file_download.php?file_id=1175&type=bug
Issue History
2014-02-20 13:24Absolute ZeroNew Issue
2014-02-20 16:23ZalewaNote Added: 0008250
2014-02-21 01:35DevilHunterNote Added: 0008252
2014-03-03 21:57ZalewaAssigned To => Zalewa
2014-03-03 21:57ZalewaStatusnew => assigned
2014-03-03 22:59ZalewaTarget Version => 0.12 Beta
2014-03-04 17:36ZalewaNote Added: 0008318
2014-03-04 17:36ZalewaStatusassigned => needs review
2014-03-06 18:19ZalewaNote Added: 0008324
2014-03-06 18:19ZalewaStatusneeds review => needs testing
2014-03-07 03:11Absolute ZeroNote Added: 0008334
2014-03-07 06:28ZalewaNote Added: 0008351
2014-03-07 06:44ZalewaStatusneeds testing => feedback
2014-03-07 16:18Absolute ZeroFile Added: Zandronum
2014-03-07 16:20Absolute ZeroNote Added: 0008359
2014-03-07 16:20Absolute ZeroStatusfeedback => assigned
2014-03-07 21:56ZalewaNote Added: 0008362
2014-03-07 21:56ZalewaStatusassigned => needs review
2014-03-07 21:56ZalewaNote Edited: 0008362bug_revision_view_page.php?bugnote_id=8362#r4555
2014-03-07 21:57ZalewaNote Edited: 0008362bug_revision_view_page.php?bugnote_id=8362#r4556
2014-03-12 06:40ZalewaNote Added: 0008384
2014-03-12 06:40ZalewaStatusneeds review => needs testing
2014-03-13 00:52Absolute ZeroNote Added: 0008390
2014-03-13 06:24ZalewaStatusneeds testing => resolved
2014-03-13 06:24ZalewaFixed in Version => 0.12 Beta
2014-03-13 06:24ZalewaResolutionopen => fixed
2018-09-29 14:46WubTheCaptainStatusresolved => closed

Notes
(0008250)
Zalewa   
2014-02-20 16:23   
Yeah, I actually noticed that too just yesterday. We'll take care of that.
(0008252)
DevilHunter   
2014-02-21 01:35   
Also had this issue before. Usually, just restarting it does well.
(0008318)
Zalewa   
2014-03-04 17:36   
This one was quite difficult. The problem occurred only when the server cache file existed on the hard drive but was empty. Anyway, this commit should fix it.
(0008324)
Zalewa   
2014-03-06 18:19   
Update is available on Beta auto-update channel, please test.
(0008334)
Absolute Zero   
2014-03-07 03:11   
I did a few adjusts to the query speed and the issue persists, relevant lines of the log:

[00:07:55] Error: Master server for Zandronum: Could not fetch a new server list from the master because not enough time has past.
[00:08:05] Error: Master server for Zandronum: Connection timeout (66.228.33.44:15300).
[00:08:05] Reloading master server results from cache for Zandronum!


Settings used were the following:

Number of tries: 3
Delay between tries: 500

Batch size: 250
Batch delay: 1
(0008351)
Zalewa   
2014-03-07 06:28   
Can you go to "%APPDATA%/.doomseeker" directory and upload the "Zandronum" file that you find there? If you're using portable version this file will be located in ".doomseeker" directory directly in the program dir.
(0008359)
Absolute Zero   
2014-03-07 16:20   
Zandronum file uploaded.
(0008362)
Zalewa   
2014-03-07 21:56   
(edited on: 2014-03-07 21:57)
These four commits improve master server refresh process and should fix the problem:
1.'https://bitbucket.org/Blzut3/doomseeker/commits/0f85fd4f74979fec05b06ac1a984f1a5bb1516dc [^]'
2.'https://bitbucket.org/Blzut3/doomseeker/commits/f3d3b1784b497b8f18b6de830c33875e0beee75f [^]'
3.'https://bitbucket.org/Blzut3/doomseeker/commits/bbbfda66754380bb2057b5e436b3f72ccee130e7 [^]'
4.'https://bitbucket.org/Blzut3/doomseeker/commits/f389f8bc2d368c6ebf6f36234ab2c61b94c12ccf [^]'

(0008384)
Zalewa   
2014-03-12 06:40   
A new testing build is out on beta update channel. Please test.
(0008390)
Absolute Zero   
2014-03-13 00:52   
Working, I suggest a counter to notice the user of what is happening (status bar seems to be a proper place already). I suggest something like this:

Error: Master server for Zandronum: You are querying too fast, please wait <seconds remaining> seconds.

Shrinking the line a bit as above is suggested to be sure that people with smaller monitors will see the whole message; consider this counter a minor thing. But hey, seeing this bug fixed is a huge improvement already. Thank you!