search

kaimallea / csgo

A containerized dedicated server for Counter-Strike: Global Offensive
The Unlicense
243 stars 58 forks source link

Fix replacing env vars in config files at startup #55

Closed minusbrain closed 3 years ago

minusbrain commented 3 years ago

Previously the logic that updates autoexec.cfg and server.cfg on startup used single quotes in the call to sed. Double quotes are required to actually replace the environment variable name with its content, though.

@kaimallea Not sure why I didn't see the issue earlier, probably because it was masked by 'sed -i -n' actually emptying the whole file anyway. Sorry about this. To be honest I am wondering how this config file update logic worked for anyone before. Maybe no one else ran into these issues because it was introduced only recently?

kaimallea commented 3 years ago

@minusbrain appreciate you fixing! sorry I'm not able to directly test these