Zandronum Chat on our Discord Server Get the latest version: 3.1
Source Code
My View | View Issues | Change Log | Roadmap | Zandronum Issue Support Ranking | Rules | My Account
  

View Issue Details [ Jump to Notes ] [ Issue History ] [ Print ]
IDProjectCategoryView StatusDate SubmittedLast Update
0000993Zandronum[All Projects] Suggestionpublic2012-08-26 21:032014-06-11 21:50
Reporterjummama 
Assigned To 
PrioritylowSeverityfeatureReproducibilityhave not tried
StatusfeedbackResolutionopen 
PlatformAllOS-OS Version-
Product Version1.0 
Target VersionFixed in Version 
Summary0000993: "sv_reporthostname" or similar setting
DescriptionI think it would be a "nice-to-have" now that the code is open, to have something like "sv_reporthostname" where one could put a string in a form of "<hostname-or-ip>:[portnumber]", which the master server could then verify, and then choose to publish on condition of success, which helps by making sure every server the master gives clients such as Doomseeker are actually capable as advertised of speaking to game clients. This would probably break compatibility with the currrent skulltag protocol, but I believe it would also represent an easier way to find servers that have upgraded from Skulltag vs Servers on Zandronum, and help users verify port forwards with log messages telling people when their servers don't have working portforwards, and other odditys.
Steps To Reproduce1. Grab the source myself
2. RFC in a bug tracket
3. ????
4. Etc.

Wish me luck. Also, I'm open to related suggestions, since I plan to dig around in that area.
Attached Files

- Relationships
related to 0000992feedbackTorr Samaho Master server code makes incorrect assumptions about the source port of updates. 

-  Notes
User avatar (0004488)
Torr Samaho (administrator)
2012-08-27 20:19
edited on: 2012-08-27 20:22

 Ok, I see why you would want to publish a hostname instead of an IP, but why would you want to override the IP?

BTW: The master already verifies that a server can be reached, so you likely won't need to extend the protocol for this.
User avatar (0004489)
jummama (reporter)
2012-08-27 20:30

 I think that having a hostname reported along with a port would be nice for people hosting with dynamic dns, so that the server list on the client end can be populated with things like doomhost.dyndns.org instead of ugly-isp-assigned-host.with-an-ip-1-2-3-4.dhcp.us.wa.everett.isp.net

I did notice that the master server connects to verify already now that I've looked at it. A hostname setting like this should do a lookup on the provided hostname so it can make sure it really points at the same ip it's receiving from, and initiate a connection on the supplied port instead of the source port.
User avatar (0004490)
jummama (reporter)
2012-08-27 20:32

 As far as IP overrides, I can't see much reason for it, but if a hostname is used there, an ip would be technically valid. I think I'll set it up so that you can use the form of *:(port) to override the port and not report a host name.
User avatar (0009124)
Watermelon (developer)
2014-06-11 21:50

 Still wanted?

Issue Community Support
Only registered users can voice their support. Click here to register, or here to log in.
Supporters: No one explicitly supports this issue yet.
Opponents: No one explicitly opposes this issue yet.

- Issue History
Date Modified Username Field Change
2012-08-26 21:03 jummama New Issue
2012-08-27 20:19 Torr Samaho Note Added: 0004488
2012-08-27 20:20 Torr Samaho Relationship added related to 0000992
2012-08-27 20:21 Torr Samaho Additional Information Updated View Revisions
2012-08-27 20:22 Torr Samaho Note Edited: 0004488 View Revisions
2012-08-27 20:30 jummama Note Added: 0004489
2012-08-27 20:32 jummama Note Added: 0004490
2014-06-11 21:50 Watermelon Note Added: 0009124
2014-06-11 21:50 Watermelon Status new => feedback




Questions or other issues? Contact Us.

Links

Copyright © 2000 - 2024 MantisBT Team
Powered by Mantis Bugtracker