Tsuey / L4D2-Community-Update

Help us shape the potential future of L4D2 vanilla.
69 stars 23 forks source link

ConCommand `snd_dump_filepaths` references non-existent UNC path #356

Open hfmkwi opened 1 year ago

hfmkwi commented 1 year ago

Description

The ConCommand snd_dump_filepaths should allow clients running with the launch option -playtest to dump the list of sounds played in the current session to the path stored in ConVar snd_store_filepaths. This appears to be standard behavior for all Valve games post-Portal 2, and Valve appears to have attempted to retrofit this for L4D2.

However, as of client version 2.2.2.6, snd_dump_filepaths ignores snd_store_filepaths and always tries to read from and write to the UNC path \\fileserver\User\Morasky\L4Dsoundlogs, rendering the command useless unless one attaches a network share with this exact path.

Reproduction steps

The following is reproducible on a clean installation of Left 4 Dead 2 :

  1. Launch the cilent with the launch option -playtest.
  2. Execute snd_dump_filepaths with a valid path in the developer console.

Additional files

No response

hfmkwi commented 1 year ago

Engine interprets host in UNC path as an engine path which is undocumented behavior.

I don’t think this issue falls in the purview of this repository; closing.

Tsuey commented 1 year ago

Technically requests like this would fall under purview of the L4D2 issues over at https://github.com/ValveSoftware/Source-1-Games -- though regarding L4D2, they are far more likely to prioritize security reports over bug/feature reports. While we cannot currently fix this issue ourselves, it is still relevant to keep note of it for the future. Feel free to re-close it if you wish to post it on Valve's repo.