Skip to content

speedtest-tracker: add external IP URL and internet check hostname in .env#10078

Merged
michelroegl-brunner merged 4 commits intomainfrom
MickLesk-patch-1
Dec 29, 2025
Merged

speedtest-tracker: add external IP URL and internet check hostname in .env#10078
michelroegl-brunner merged 4 commits intomainfrom
MickLesk-patch-1

Conversation

@MickLesk
Copy link
Copy Markdown
Member

@MickLesk MickLesk commented Dec 17, 2025

✍️ Description

🔗 Related Issue

Fixes #10415

✅ Prerequisites (X in brackets)

  • Self-review completed – Code follows project standards.
  • Tested thoroughly – Changes work as expected.
  • No security risks – No hardcoded secrets, unnecessary privilege escalations, or permission issues.

🛠️ Type of Change (X in brackets)

  • 🐞 Bug fix – Resolves an issue without breaking functionality.
  • New feature – Adds new, non-breaking functionality.
  • 💥 Breaking change – Alters existing functionality in a way that may require updates.
  • 🆕 New script – A fully functional and tested script or script set.
  • 🌍 Website update – Changes to website-related JSON files or metadata.
  • 🔧 Refactoring / Code Cleanup – Improves readability or maintainability without changing functionality.
  • 📝 Documentation update – Changes to README, AppName.md, CONTRIBUTING.md, or other docs.

@MickLesk MickLesk requested a review from a team as a code owner December 17, 2025 15:03
@github-actions github-actions Bot added feature update script A change that updates a script labels Dec 17, 2025
@MickLesk MickLesk marked this pull request as draft December 17, 2025 15:11
@MickLesk
Copy link
Copy Markdown
Member Author

testing...

CrazyWolf13
CrazyWolf13 previously approved these changes Dec 17, 2025
@Dunky13
Copy link
Copy Markdown

Dunky13 commented Dec 18, 2025

Adding these env variables to /opt/speedtest-tracker/.env did not resolve the issue for me. Settings were applied

@MickLesk
Copy link
Copy Markdown
Member Author

Yeah i know, thats the reason for Draft.

@CrazyWolf13
Copy link
Copy Markdown
Member

Possibly we can add this as well @MickLesk

# Fix Ping issues: https://github.com/alexjustesen/speedtest-tracker/issues/2525#issuecomment-3677992174
setcap cap_net_raw+p /bin/ping

At least for me this solved some of the issues.

@CrazyWolf13
Copy link
Copy Markdown
Member

maybe we should consider using https://ip.me when using curl ip.me this also just returns the public IP, and I've been using this for years, the adress https://ipecho.net/plain is also in the same hazegi dns blocklist as the initial domain used by speedtest-tracker by default, so this does not add any value in that regard

@MickLesk
Copy link
Copy Markdown
Member Author

Doesnt Work too 😂

@CrazyWolf13
Copy link
Copy Markdown
Member

for me it does :D
What dns blocklist blocks it?

@drVLecter
Copy link
Copy Markdown

For me none of the changes in env file worked.

What did work is following command/s within the speedtracker lxc:

setcap cap_net_raw+ep /bin/ping && 
sudo -u www-data ping -c 1 icanhazip.com && 
systemctl restart speedtest-tracker

This worked on both old working but broken after update lxc or a new up-to-date lxc that didnt work out of the box.

@MickLesk MickLesk marked this pull request as ready for review December 29, 2025 18:43
@michelroegl-brunner michelroegl-brunner merged commit 2cc54fd into main Dec 29, 2025
2 checks passed
@MickLesk MickLesk deleted the MickLesk-patch-1 branch January 13, 2026 07:56
@github-actions github-actions Bot locked as resolved and limited conversation to collaborators Feb 3, 2026
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

bugfix feature update script A change that updates a script

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Speedtest-Tracker requires cap_net_raw+p on /usr/bin/ping in versions >= 1.13.0

6 participants