0003202: Dynamic light size cannot be enforced by server

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

View Issue Details [ Jump to Notes ]

[ Issue History ] [ Print ]

Project

Category

View Status

Date Submitted

Last Update

0003202

Zandronum

[All Projects] Suggestion

public

2017-07-27 17:33

2020-12-09 21:56

Reporter

Korshun

Assigned To

Priority

normal

Severity

minor

Reproducibility

always

Status

needs testing

Resolution

open

Platform

Microsoft

Windows

OS Version

XP/Vista/7

Product Version

3.0-beta

Target Version

3.1

Fixed in Version

Summary

0003202: Dynamic light size cannot be enforced by server

Description

Setting gl_lights_size to something huge, like 10, can be used to see what's happening behind walls, and is not enforced by sv_forcegldefaults.

Additional Information

Considering that even the default size of 1 can be theoretically used as a wallhack, it might be preferable to introduce a new cvar to force light size (for maps that require them) or disable dynamic lights. Something like sv_maxlightsize with a default of -1 (unlimited), 0 being force off, 1 being no larger than default size.

Attached Files

Relationships

has duplicate

0003222

closed

opengl -> glowing object glows via opening walls

Notes
(0018110) jdagenet (reporter) 2017-07-28 00:45	I think it'd be better and easier to just force them on with a size of 1 if SV_ForceGLDefaults is true. Seems pointless to have a separate server variable.

(0018175) Dusk (developer) 2017-08-13 10:39	I agree, this should be tacked onto sv_forcegldefaults.

(0018190) Filystea (reporter) 2017-08-14 18:38	since my ticket was closed due to duplicate i clicked strong support tho i think your issue is different.

(0019218) StrikerMan780 (reporter) 2018-05-07 20:09 edited on: 2018-05-07 20:10	In the latest GZDoom, the ability to change dynamic light size was removed. (And for good reason.) Might as well follow that.

(0021578) doomjoshuaboy (developer) 2020-12-09 21:56	This was fixed in zandronum but need further testing. Thanks to Kaminsky.

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

Issue History
Date Modified	Username	Field	Change
2017-07-27 17:33	Korshun	New Issue
2017-07-28 00:45	jdagenet	Note Added: 0018110
2017-08-13 10:36	Dusk	Relationship added	has duplicate 0003222
2017-08-13 10:39	Dusk	Note Added: 0018175
2017-08-13 10:39	Dusk	Target Version	=> 3.1
2017-08-14 18:38	Filystea	Note Added: 0018190
2018-05-07 20:09	StrikerMan780	Note Added: 0019218
2018-05-07 20:10	StrikerMan780	Note Edited: 0019218	View Revisions
2020-12-09 21:56	doomjoshuaboy	Note Added: 0021578
2020-12-09 21:56	doomjoshuaboy	Status	new => needs testing

Questions or other issues? Contact Us.

Links

Doom and Doom II are the property of id Software.