[GH-ISSUE #3876] [BUG] The scripts using grep to get game server information can garble info when multiple lines match the grep. #2521

Closed
opened 2026-02-27 03:03:32 +03:00 by kerem · 3 comments
Owner

Originally created by @Avanesov79 on GitHub (May 29, 2022).
Original GitHub issue: https://github.com/GameServerManagers/LinuxGSM/issues/3876

User Story

As a server admin I want the game server gathering scripts to not assume that the target string is only listed once in the config file.

Basic info

Distro: AlmaLinux 8.6 (Sky Tiger)
Game: Ark: Survival Evolved
Command: I have noticed this in details, backup and I believe it is responsible for my discord alert failure as well.
LinuxGSM version: v21.5.1

Further Information

I use the Beacon server manager to setup my Ark configuration files. It prints the config files with additional lines that contains meta data about what fields it manages. When the LinuxGSM scripts gathers info from these files it grabs the additional lines that also contains references to the keys it is looking for and outputs it all together. For example when I run "./arklostisland details" the "Server Name:" section lists:

Server name:         ManagedKeys(SectionSessionSettingsKeys())
Avas Lost Island Ark Server

This is because the "SessionName" string that is grepped is also included in the [Beacon] section so Beacon knows it is a managed key.

To Reproduce

Steps to reproduce the behaviour:

  1. Add additional lines to a config file that is grepped for a specific string and reference that string along with other information.
  2. Run ./serverscript details (or backup, or run a discord test-alert if the key for the server name is referenced...it will not parse the JSON)
  3. check your output

Expected behaviour

The information gathering scripts should be able to distinguish between lines that contain only the needed key=value pair and other lines that may reference the desired key.

I am attaching my ark server config file that contains the added content by Beacon (private info redacted) as well as a file with relevant output from the details command to show what is gathered. The relevant details fields are: Server Name, Server Password and Admin Password.
details_info.txt
GameUserSettings_upload.ini.txt

Originally created by @Avanesov79 on GitHub (May 29, 2022). Original GitHub issue: https://github.com/GameServerManagers/LinuxGSM/issues/3876 ## User Story As a server admin I want the game server gathering scripts to not assume that the target string is only listed once in the config file. ## Basic info Distro: AlmaLinux 8.6 (Sky Tiger) Game: Ark: Survival Evolved Command: I have noticed this in details, backup and I believe it is responsible for my discord alert failure as well. LinuxGSM version: v21.5.1 ## Further Information I use the Beacon server manager to setup my Ark configuration files. It prints the config files with additional lines that contains meta data about what fields it manages. When the LinuxGSM scripts gathers info from these files it grabs the additional lines that also contains references to the keys it is looking for and outputs it all together. For example when I run "./arklostisland details" the "Server Name:" section lists: ``` Server name: ManagedKeys(SectionSessionSettingsKeys()) Avas Lost Island Ark Server ``` This is because the "SessionName" string that is grepped is also included in the [Beacon] section so Beacon knows it is a managed key. ## To Reproduce Steps to reproduce the behaviour: 1. Add additional lines to a config file that is grepped for a specific string and reference that string along with other information. 2. Run ./serverscript details (or backup, or run a discord test-alert if the key for the server name is referenced...it will not parse the JSON) 3. check your output ## Expected behaviour The information gathering scripts should be able to distinguish between lines that contain only the needed key=value pair and other lines that may reference the desired key. I am attaching my ark server config file that contains the added content by Beacon (private info redacted) as well as a file with relevant output from the details command to show what is gathered. The relevant details fields are: Server Name, Server Password and Admin Password. [details_info.txt](https://github.com/GameServerManagers/LinuxGSM/files/8792517/details_info.txt) [GameUserSettings_upload.ini.txt](https://github.com/GameServerManagers/LinuxGSM/files/8792519/GameUserSettings_upload.ini.txt)
Author
Owner

@h3o66 commented on GitHub (May 29, 2022):

See the connected pr #3878 this hopefully fixes the issue for you and probably others.

<!-- gh-comment-id:1140526393 --> @h3o66 commented on GitHub (May 29, 2022): See the connected pr #3878 this hopefully fixes the issue for you and probably others.
Author
Owner

@Avanesov79 commented on GitHub (May 31, 2022):

I manually applied the changes in #3878 and it has fixed the issue for me. The details command now displays all fields correctly and test-alert no longer fails to parse JSON for discord.

<!-- gh-comment-id:1142177412 --> @Avanesov79 commented on GitHub (May 31, 2022): I manually applied the changes in [#3878](https://github.com/GameServerManagers/LinuxGSM/pull/3878) and it has fixed the issue for me. The details command now displays all fields correctly and test-alert no longer fails to parse JSON for discord.
Author
Owner

@github-actions[bot] commented on GitHub (Jun 4, 2023):

This issue has been automatically locked since there has not been any recent activity after it was closed. Please open a new issue for related bugs.

<!-- gh-comment-id:1575275601 --> @github-actions[bot] commented on GitHub (Jun 4, 2023): This issue has been automatically locked since there has not been any recent activity after it was closed. Please open a new issue for related bugs.
Sign in to join this conversation.
No labels
Atomic
Epic
cannot reproduce
command: backup
command: console
command: debug
command: details
command: fast-dl
command: install
command: mods
command: monitor
command: post-details
command: restart
command: send
command: start
command: stop
command: update
command: update-lgsm
command: validate
command: wipe
distro: AlmaLinux
distro: Arch Linux
distro: CentOS
distro: Debian
distro: Fedora
distro: RedHat
distro: Rocky Linux
distro: Ubuntu
distro: openSUSE
engine: goldsrc
engine: source
game: 7 Days to Die
game: ARMA 3
game: Ark: Survival Evolved
game: Assetto Corsa
game: Avorion
game: BATTALION: Legacy
game: Barotrauma
game: Battalion 1944
game: Battlefield 1942
game: Black Mesa: Deathmatch
game: Blade Symphony
game: Call of Duty 2
game: Call of Duty 4
game: Call of Duty: United Offensive
game: Counter-Strike 1.6
game: Counter-Strike 2
game: Counter-Strike: Global Offensive
game: Counter-Strike: Source
game: Day of Infamy
game: Dayz
game: Death Match Classic
game: Don't Starve Together
game: ET: Legacy
game: Eco
game: Factorio
game: Factorio
game: Garry's Mod
game: Half-Life
game: Hurtword
game: Insurgecy
game: Insurgecy
game: Insurgency: Sandstorm
game: Just Cause 3
game: Killing Floor
game: Killing Floor 2
game: Left 4 Dead 2
game: Minecraft
game: Minecraft Bedrock
game: Mordhau
game: Multi Theft Auto
game: Mumble
game: Natural Selection 2
game: No More Room in Hell
game: Pavlov VR
game: Post Scriptum
game: Project Zomboid
game: Quake 3
game: QuakeWorld
game: Red Orchestra: Ostfront 41-45
game: Return to Castle Wolfenstein
game: Rising World
game: Rust
game: San Andreas Multiplayer
game: Satisfactory
game: Soldat
game: Soldier of Fortune 2
game: Squad
game: Squad 44
game: Starbound
game: Stationeers
game: Sven Co-op
game: Team Fortress 2
game: Teamspeak 3
game: Teeworlds
game: Terraria
game: The Front
game: Unreal Tournament 2004
game: Unreal Tournament 3
game: Unreal Tournament 99
game: Unturned
game: Valheim
game: Wurm Unlimited
game: Zombie Master Reborn
game: label missing
good first issue
help wanted
info: alerts
info: dependency
info: docker
info: docs
info: email
info: query
info: steamcmd
info: systemd
info: tmux
info: website
info: website
needs more info
outcome: duplicate
outcome: issue resolved
outcome: issue resolved
outcome: issue unresolved
outcome: pr accepted
outcome: pr rejected
outcome: unconfirmed
outcome: wontfix
outcome: wrong forum
potential-duplicate
priority
pull-request
type: bug
type: feature
type: feature
type: feature request
type: game server request
type: refactor
waiting response
No milestone
No project
No assignees
1 participant
Notifications
Due date
The due date is invalid or out of range. Please use the format "yyyy-mm-dd".

No due date set.

Dependencies

No dependencies set.

Reference
starred/LinuxGSM#2521
No description provided.