[PR #2789] [MERGED] fix(ts3server): improve ts3 ip parsing #3786

Closed
opened 2026-02-27 03:55:48 +03:00 by kerem · 0 comments
Owner

📋 Pull Request Information

Original PR: https://github.com/GameServerManagers/LinuxGSM/pull/2789
Author: @GrillGuth
Created: 3/27/2020
Status: Merged
Merged: 3/27/2020
Merged by: @dgibbs64

Base: developHead: patch-1


📝 Commits (2)

  • 9e53770 Merge branch 'hotfix/v20.1.5'
  • 0feb9cb Use only first IP for TS3-servers

📊 Changes

1 file changed (+1 additions, -1 deletions)

View changed files

📝 lgsm/functions/info_config.sh (+1 -1)

📄 Description

For Teamspeak3-Servers: Only use the first IP specified within the ts3server.ini file. The IP string is cut off after the first appearance of a comma (IP seperator within ts3server.ini files).

Description

As described inte teamspeak server documentation the voice_ip parameter allows a comma separated IP list. Currently when e.g. "voice_ip=12.345.678.910, 1a23:b456:7891:011::1" is specified which is according to ts standards LinuxGSM will extract "12.345.678.910 1a23b45678910111" as IP string and can not deal with it. As a workaround till more then one IP and IPv6 is supported the IP string should be cut off after the first appearance of a comma.

Fixes #2658

Type of change

  • Bug fix (change which fixes an issue).

Checklist

PR will not be merged until all steps are complete.

  • This pull request links to an issue.
  • This pull request uses the develop branch as its base.
  • This pull request Subject follows the Conventional Commits standard.
  • This code follows the style guidelines of this project.
  • I have performed a self-review of my own code.
  • I have checked that this code is commented where required.
  • I have provided a detailed enough description of this PR.
  • I have checked If documentation needs updating.

Documentation

If documentation does need updating either update it by creating a PR (preferred) or request a documentation update.

Thank you for your Pull Request!


🔄 This issue represents a GitHub Pull Request. It cannot be merged through Gitea due to API limitations.

## 📋 Pull Request Information **Original PR:** https://github.com/GameServerManagers/LinuxGSM/pull/2789 **Author:** [@GrillGuth](https://github.com/GrillGuth) **Created:** 3/27/2020 **Status:** ✅ Merged **Merged:** 3/27/2020 **Merged by:** [@dgibbs64](https://github.com/dgibbs64) **Base:** `develop` ← **Head:** `patch-1` --- ### 📝 Commits (2) - [`9e53770`](https://github.com/GameServerManagers/LinuxGSM/commit/9e5377086bbbb0bc065c5a8990b25c3d472c1d20) Merge branch 'hotfix/v20.1.5' - [`0feb9cb`](https://github.com/GameServerManagers/LinuxGSM/commit/0feb9cbb2d45f297fe7dc50c97f8575041d8275e) Use only first IP for TS3-servers ### 📊 Changes **1 file changed** (+1 additions, -1 deletions) <details> <summary>View changed files</summary> 📝 `lgsm/functions/info_config.sh` (+1 -1) </details> ### 📄 Description For Teamspeak3-Servers: Only use the first IP specified within the ts3server.ini file. The IP string is cut off after the first appearance of a comma (IP seperator within ts3server.ini files). # Description As described inte teamspeak server documentation the voice_ip parameter allows a comma separated IP list. Currently when e.g. "voice_ip=12.345.678.910, 1a23:b456:7891:011::1" is specified which is according to ts standards LinuxGSM will extract "12.345.678.910 1a23b45678910111" as IP string and can not deal with it. As a workaround till more then one IP and IPv6 is supported the IP string should be cut off after the first appearance of a comma. Fixes #2658 ## Type of change * [x] Bug fix (change which fixes an issue). ## Checklist PR will not be merged until all steps are complete. * [x] This pull request links to an issue. * [x] This pull request uses the `develop` branch as its base. * [x] This pull request Subject follows the Conventional Commits standard. * [x] This code follows the style guidelines of this project. * [x] I have performed a self-review of my own code. * [x] I have checked that this code is commented where required. * [x] I have provided a detailed enough description of this PR. * [x] I have checked If documentation needs updating. ## Documentation If documentation does need updating either update it by creating a PR (preferred) or request a documentation update. * User docs: https://github.com/GameServerManagers/LinuxGSM-Docs * Dev docs: https://github.com/GameServerManagers/LinuxGSM-Dev-Docs **Thank you for your Pull Request!** --- <sub>🔄 This issue represents a GitHub Pull Request. It cannot be merged through Gitea due to API limitations.</sub>
kerem 2026-02-27 03:55:48 +03:00
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#3786
No description provided.