[PR #50] [CLOSED] Ping and Traceroute: Review wanted #56

New issue

Closed

opened 2026-02-25 21:30:58 +03:00 by kerem · 0 comments

kerem commented

2026-02-25 21:30:58 +03:00

Owner

📋 Pull Request Information

Original PR: https://github.com/charlesabarnes/SPFtoolbox/pull/50
Author: @charlesabarnes
Created: 10/13/2021
Status: ❌ Closed

Base: master ← Head: network-tools

📝 Commits (10+)

7ad733c add ping to spf toolbox
415d7d5 update class name
6ae2704 format whitespace
96f0324 use proper variable
9275736 fix html
624be65 change directory
4d4bee8 add traceroute
8ed0604 add info to frontend
e5c13f6 remove semicolon that is causing error
0e4926d update variable name in traceroute

📊 Changes

5 files changed (+59 additions, -0 deletions)

View changed files

📝 README.md (+2 -0)
📝 index.php (+10 -0)
➕ operations/Ping.php (+15 -0)
➕ operations/Traceroute.php (+24 -0)
📝 operations/index.php (+8 -0)

📄 Description

Ping and Traceroute has to be installed on the associated machine. I am thinking about hitting a fallback server. any complaints with getting the response from https://spftoolbox.com as a fallback?

_{🔄 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/charlesabarnes/SPFtoolbox/pull/50 **Author:** [@charlesabarnes](https://github.com/charlesabarnes) **Created:** 10/13/2021 **Status:** ❌ Closed **Base:** `master` ← **Head:** `network-tools` --- ### 📝 Commits (10+) - [`7ad733c`](https://github.com/charlesabarnes/SPFtoolbox/commit/7ad733c73e5d4340ba7656d4bec3e6680204d381) add ping to spf toolbox - [`415d7d5`](https://github.com/charlesabarnes/SPFtoolbox/commit/415d7d59caf669e22e7e0de67391de4dd86b5acb) update class name - [`6ae2704`](https://github.com/charlesabarnes/SPFtoolbox/commit/6ae2704eb10cce4961222f442309ee66cb24c062) format whitespace - [`96f0324`](https://github.com/charlesabarnes/SPFtoolbox/commit/96f03247d49477c9bb32f297959ebcfb362745e0) use proper variable - [`9275736`](https://github.com/charlesabarnes/SPFtoolbox/commit/92757366b1c17a95a0b1e3e08a9cd7daa0ee671e) fix html - [`624be65`](https://github.com/charlesabarnes/SPFtoolbox/commit/624be659b4e9aa63e109a789d6617622e79d5bfa) change directory - [`4d4bee8`](https://github.com/charlesabarnes/SPFtoolbox/commit/4d4bee8d52f21a8e3262dfa5c7c7ae5f8970db1f) add traceroute - [`8ed0604`](https://github.com/charlesabarnes/SPFtoolbox/commit/8ed06042bce7451f714d9c6e2d217899a2f48581) add info to frontend - [`e5c13f6`](https://github.com/charlesabarnes/SPFtoolbox/commit/e5c13f6c24f54f0b75eff7f44ed35e5bd4f981ca) remove semicolon that is causing error - [`0e4926d`](https://github.com/charlesabarnes/SPFtoolbox/commit/0e4926d61c26c6585ce4a6e0185ee9b667bac350) update variable name in traceroute ### 📊 Changes **5 files changed** (+59 additions, -0 deletions) <details> <summary>View changed files</summary> 📝 `README.md` (+2 -0) 📝 `index.php` (+10 -0) ➕ `operations/Ping.php` (+15 -0) ➕ `operations/Traceroute.php` (+24 -0) 📝 `operations/index.php` (+8 -0) </details> ### 📄 Description Ping and Traceroute has to be installed on the associated machine. I am thinking about hitting a fallback server. any complaints with getting the response from https://spftoolbox.com as a fallback? --- <sub>🔄 This issue represents a GitHub Pull Request. It cannot be merged through Gitea due to API limitations.</sub>