Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Add speedtest-tracker. #379

Merged
merged 7 commits into from
Jan 25, 2025
Merged

Add speedtest-tracker. #379

merged 7 commits into from
Jan 25, 2025

Conversation

mcmikemn
Copy link
Collaborator

Add Speedtest Tracker.

@mcmikemn mcmikemn self-assigned this Dec 23, 2024
@mcmikemn mcmikemn added the enhancement New feature or request label Dec 23, 2024
@mcmikemn
Copy link
Collaborator Author

I'm trying to add Speedtest Tracker to see if it's a tool I want to use. It looks like it should be simple to dry-ize but Traefik is only using the Traefik default cert, even though the app seems to be configured correctly:
image

The logs say "generating self-signed keys in /config/keys, you can replace these with your own keys if required". My guess is that it's generating its own keys for its own nginx, and it wouldn't have any effect on Traefik's certs. But just in case, I manually replaced the self-signed keys with the cert and key from Traefik and then restarted the container. It didn't work.

Do you have suggestions, @EnigmaCurry, what I might be forgetting or should look at?

speedtest-tracker/traefik-cert.pem Outdated Show resolved Hide resolved
speedtest-tracker/traefik-key.pem Outdated Show resolved Hide resolved
@EnigmaCurry
Copy link
Owner

EnigmaCurry commented Dec 23, 2024

The logs say "generating self-signed keys in /config/keys, you can replace these with your own keys if required".

If it says that in the app log (not the traefik log) then it seems like the app is trying to do its own TLS. you just need to turn that off and let traefik be the only thing that does TLS. The app itself should only be handling unencrypted normal HTTP.

@mcmikemn
Copy link
Collaborator Author

Everything works now. Ready for review.

@mcmikemn mcmikemn requested a review from EnigmaCurry January 23, 2025 20:37
@EnigmaCurry EnigmaCurry marked this pull request as ready for review January 23, 2025 21:50
speedtest-tracker/Makefile Outdated Show resolved Hide resolved
speedtest-tracker/Makefile Outdated Show resolved Hide resolved
@EnigmaCurry EnigmaCurry merged commit 0bc1ed2 into master Jan 25, 2025
@EnigmaCurry EnigmaCurry deleted the speedtest-tracker branch January 25, 2025 20:32
@EnigmaCurry
Copy link
Owner

I love that these just happen to be listed alphabetically together

Smokeping - a network latency measurement tool
Speedtest Tracker - a network performance monitor

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants