MantisBT - Zandronum
View Issue Details
0001517Zandronum[All Projects] Suggestionpublic2013-09-23 16:222019-06-04 21:59
Dusk 
 
normaltweakN/A
closedno change required 
 
 
0001517: improving outlook of network errors
- main point of this: the client processes auth failure wad data further and shows what wads differ (a bit like how this stuff is dealt with demos)
- making network errors appear separate and orange to make the error clearer (and cleaned up the code a teeny bit while at that)
No tags attached.
related to 0001457closed Torr Samaho Extended diagnostics for level auth failures 
diff neterrors.diff (14,302) 2013-09-23 19:44
/tracker/file_download.php?file_id=1069&type=bug
Issue History
2013-09-23 16:22DuskNew Issue
2013-09-23 16:22DuskStatusnew => assigned
2013-09-23 16:22DuskAssigned To => Dusk
2013-09-23 17:44DuskNote Added: 0007273
2013-09-23 17:46DuskFile Added: neterrors.diff
2013-09-23 17:47DuskNote Edited: 0007273bug_revision_view_page.php?bugnote_id=7273#r4073
2013-09-23 18:35ArcoNote Added: 0007274
2013-09-23 18:51DuskFile Deleted: neterrors.diff
2013-09-23 18:51DuskFile Added: neterrors.diff
2013-09-23 18:51DuskStatusassigned => needs review
2013-09-23 18:52DuskNote Added: 0007275
2013-09-23 19:43DuskFile Deleted: neterrors.diff
2013-09-23 19:44DuskFile Added: neterrors.diff
2013-09-26 19:54Torr SamahoNote Added: 0007280
2013-09-27 10:33DuskNote Added: 0007284
2013-09-27 10:33DuskStatusneeds review => acknowledged
2013-09-29 07:30Torr SamahoNote Added: 0007297
2013-10-08 01:53ArcoRelationship addedrelated to 0001457
2014-06-06 18:47DuskResolutionopen => suspended
2014-06-06 18:47DuskTarget Version1.3 =>
2019-06-04 21:59DuskAssigned ToDusk =>
2019-06-04 21:59DuskStatusacknowledged => closed
2019-06-04 21:59DuskResolutionsuspended => no change required

Notes
(0007273)
Dusk   
2013-09-23 17:44   
(edited on: 2013-09-23 17:47)
I got this actually working now. I attached a diff of what I have right now though I'm still performing testing on this.



(0007274)
Arco   
2013-09-23 18:35   
Would it be possible to also show the hashes for the wads loaded?
(0007275)
Dusk   
2013-09-23 18:52   
I guess.. not sure if they should show though.
(0007280)
Torr Samaho   
2013-09-26 19:54   
It should definitely show the md5sums of conflicting wads.

One thing your check doesn't cover is the correct order of the wads. From a quick glance at the diff it looks to me as if your patch will output that everything matches if client and server load exactly the same wads, but in a different order. In the old raw output you could see whether the order was correct.
(0007284)
Dusk   
2013-09-27 10:33   
Guess I will have to rethink this one then.
(0007297)
Torr Samaho   
2013-09-29 07:30   
You could also leave it as it is and just re-add the full dump at the end. This way the client still get's all the information.