0001947: authhostname is only read on bootup

Chat on our Discord Server Get the latest version: 3.1
Source Code

View Issue Details [ Jump to Notes ]

[ Issue History ] [ Print ]

Project

Category

View Status

Date Submitted

Last Update

0001947

Zandronum

[All Projects] Bug

public

2014-10-06 21:56

2014-10-07 19:26

Reporter

AlexMax

Assigned To

Priority

normal

Severity

minor

Reproducibility

always

Status

new

Resolution

open

Platform

OS Version

Product Version

1.3

Target Version

Fixed in Version

Summary

0001947: authhostname is only read on bootup

Description

Pretty much what it says on the tin - authhostname is only parsed once on server boot and never afterwards.

Is this a bug or a feature? Seems to me like switching authentication servers - at least right now - is a BAD idea due to the fact that usernames in one auth server might not line up with usernames on another. However, this fact is never revealed to server administrators, who can change the CVAR at will and might chalk up its current behavior to a bug.

Does ZDoom have a concept of a read-only CVAR? Perhaps it could be 'locked in' after startup to ensure nobody tries anything tricky?

Attached Files

Relationships

Notes
(0010372) Dusk (developer) 2014-10-07 00:23	Perhaps the authhostname should become a command-line parameter and the CVAR replaced with a console command which prints the authhostname?

(0010379) Torr Samaho (administrator) 2014-10-07 16:57	When I implemented this, I thought it's fine if changing the CVAR requires a restart, but completely forgot to document this anywhere. The easiest (and a quite secure) solution would be to keep the current behavior, but print a warning to the console that the changed auth server will only be used after the server is restarted. I prefer a CVAR over a command-line parameter because it allows us to transparently set a default auth server, which we already do for the master server.

**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
2014-10-06 21:56	AlexMax	New Issue
2014-10-07 00:23	Dusk	Note Added: 0010372
2014-10-07 16:57	Torr Samaho	Note Added: 0010379

Questions or other issues? Contact Us.

Links

Doom and Doom II are the property of id Software.