[PR #4332] [MERGED] feat: convert public ip to json #4308

Closed
opened 2026-02-27 04:54:39 +03:00 by kerem · 0 comments
Owner

📋 Pull Request Information

Original PR: https://github.com/GameServerManagers/LinuxGSM/pull/4332
Author: @dgibbs64
Created: 10/8/2023
Status: Merged
Merged: 10/8/2023
Merged by: @dgibbs64

Base: developHead: feature/public-ip


📝 Commits (7)

  • 913622c refactor: improve retrieval and handling of public IP address
  • da663e9 refactor: update API URL and set default values for public IP, country, and country code
  • 5029b3c feat: cache and retrieve public IP address
  • b2e6d07 feat: cache and retrieve public IP address
  • c0d2648 Merge remote-tracking branch 'origin/feature/public-ip' into feature/public-ip
  • ff56bab refactor: improve caching and logging of public IP address
  • feddfbf refactor(info_messages): Remove Mailgun alert

📊 Changes

2 files changed (+18 additions, -17 deletions)

View changed files

📝 lgsm/modules/info_game.sh (+18 -14)
📝 lgsm/modules/info_messages.sh (+0 -3)

📄 Description

Description

updated the public ip provider to http://ip-api.com.json that can provide more detail about a servers public ip. This will allow alerts to in future provide more details

Fixes #[issue]

Type of change

  • Bug fix (a change which fixes an issue).
  • New feature (a change which adds functionality).
  • New Server (new server added).
  • Refactor (restructures existing code).
  • Comment update (typo, spelling, explanation, examples, etc).

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 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/4332 **Author:** [@dgibbs64](https://github.com/dgibbs64) **Created:** 10/8/2023 **Status:** ✅ Merged **Merged:** 10/8/2023 **Merged by:** [@dgibbs64](https://github.com/dgibbs64) **Base:** `develop` ← **Head:** `feature/public-ip` --- ### 📝 Commits (7) - [`913622c`](https://github.com/GameServerManagers/LinuxGSM/commit/913622c02aed86de19c78107fc7aba5b3f049e4d) refactor: improve retrieval and handling of public IP address - [`da663e9`](https://github.com/GameServerManagers/LinuxGSM/commit/da663e996afff24a1614b5e97a9d9b4cb60f9a55) refactor: update API URL and set default values for public IP, country, and country code - [`5029b3c`](https://github.com/GameServerManagers/LinuxGSM/commit/5029b3cd877a0a68ae368d2489a15af7f8ecf2d1) feat: cache and retrieve public IP address - [`b2e6d07`](https://github.com/GameServerManagers/LinuxGSM/commit/b2e6d07fa61b5dec2baacbe8a4836620f309b6a9) feat: cache and retrieve public IP address - [`c0d2648`](https://github.com/GameServerManagers/LinuxGSM/commit/c0d2648adc6907a6ed641c78dd7e4ddeb696b774) Merge remote-tracking branch 'origin/feature/public-ip' into feature/public-ip - [`ff56bab`](https://github.com/GameServerManagers/LinuxGSM/commit/ff56bab0eb5ff07ca5b0bd0fcb89afea1e78c1c3) refactor: improve caching and logging of public IP address - [`feddfbf`](https://github.com/GameServerManagers/LinuxGSM/commit/feddfbf49c95859747c97c2251ef247ea1789484) refactor(info_messages): Remove Mailgun alert ### 📊 Changes **2 files changed** (+18 additions, -17 deletions) <details> <summary>View changed files</summary> 📝 `lgsm/modules/info_game.sh` (+18 -14) 📝 `lgsm/modules/info_messages.sh` (+0 -3) </details> ### 📄 Description # Description updated the public ip provider to http://ip-api.com.json that can provide more detail about a servers public ip. This will allow alerts to in future provide more details Fixes #[issue] ## Type of change - [ ] Bug fix (a change which fixes an issue). - [ ] New feature (a change which adds functionality). - [ ] New Server (new server added). - [x] Refactor (restructures existing code). - [ ] Comment update (typo, spelling, explanation, examples, etc). ## 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 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. - 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 04:54:39 +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#4308
No description provided.