-
-
Notifications
You must be signed in to change notification settings - Fork 566
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
Fixes formatting of unknown host in DHCP leases table #1509
Conversation
Signed-off-by: yubiuser <[email protected]>
This pull request has been mentioned on Pi-hole Userspace. There might be relevant details there: https://discourse.pi-hole.net/t/unknown-clients-comes-up-as-i-unknown-i-for-dhcp-leases/35663/13 |
This pull request has been mentioned on Pi-hole Userspace. There might be relevant details there: https://discourse.pi-hole.net/t/dhcp-leases-show-up-as-i-unknown-i/38876/2 |
This pull request has been mentioned on Pi-hole Userspace. There might be relevant details there: https://discourse.pi-hole.net/t/how-to-apply-unknown-host-in-dhcp-leases-table-from-github/40027/1 |
This pull request has been mentioned on Pi-hole Userspace. There might be relevant details there: https://discourse.pi-hole.net/t/pi-hole-core-web-v5-2-and-ftl-v5-3-released/40909/1 |
Hello, I'm still seeing this in Pi-hole v5.8.1 FTL v5.13 Web Interface v5.10.1. Has there been a regression of some sort? |
Maybe this line is causing that? https://github.com/pi-hole/AdminLTE/blob/master/settings.php#L680 |
This should be fixed by #2100 which was part of the latest release. |
By submitting this pull request, I confirm the following: ]}`
git rebase
)git commit --signoff
)What does this PR aim to accomplish?:
Fixes an issue raised on discourse https://discourse.pi-hole.net/t/unknown-clients-comes-up-as-i-unknown-i-for-dhcp-leases/35663 where unknown clients in the DHCP lease table are shown as
<i>unknown</i>
Fixes #1513, Fixes #1524, Fixes #753
How does this PR accomplish the above?:
Uses
htmlentities
on host(name) before introducing formatting code for unknown hosts and removeshtmlentities
when string is printed.