(SetupChecks) Add the actual hostname/URL/IP that triggered the warning/error
Nobody has claimed this yet.
- Dominant language
- PHP
- Stars
- 36.9k
- Forks
- 5.2k
- Avg merge
- 2d 3h
- Merged PRs (30d)
- 713
Description
Where possible, we should include the full URL and/or hostname (along with associated IP address) that triggered a setup check error or warning.
Should help ease troubleshooting for admins and reduce false bug reports and unnecessary support inquiries.
Applicable to any setup checks that are URL or hostname driven.
Would help people seeing these messages due to:
- long forgotten hostname/URL/IP address entries in their configs
- their internal (or instance-level) DNS resolution for a given hostname being different from external DNS.
e.g.
Output in the message:
- the full URL tested that triggered the error
- the IP address that the hostname in the URL resolves (from the perspective of the instance)
Contributor guide
First steps
- Read the whole issue, then the project's contributing guide.
- Comment on the issue to say you are picking it up — it saves two people doing the same work.
- Fork the repository and make your change on a branch.
- Open a pull request that references the issue number.
Research direction
Start by locating setup checks that test URLs or hostnames and inspect how their warnings and errors are assembled. The work is done when applicable messages include the full tested URL or hostname and the IP resolved from the instance perspective, with coverage across the relevant URL-driven checks.
Written by the indexing model from the issue text.
Assessment
- Tech stack
- php
- Domain
- backend, networking
- Issue type
- Feature
- Difficulty
- 4/5
- Estimated time
- 3-5 days
- Activity status
- Stale
- Clarity
- Mostly clear
- Newbie friendliness
- 38/100