Skip to content

Commit

Permalink
Make second argument of resolveHostname() optional
Browse files Browse the repository at this point in the history
Signed-off-by: DL6ER <dl6er@dl6er.de>
  • Loading branch information
DL6ER committed Apr 4, 2018
1 parent 6e99e1d commit 71a7c84
Show file tree
Hide file tree
Showing 2 changed files with 2 additions and 2 deletions.
2 changes: 1 addition & 1 deletion api_FTL.php
Expand Up @@ -309,7 +309,7 @@
foreach($return as $line)
{
$tmp = explode(" ",$line);
$forward_dest[resolveHostname($tmp[2])] = floatval($tmp[1]);
$forward_dest[resolveHostname($tmp[2],true)] = floatval($tmp[1]);
}

$result = array('clients' => $forward_dest);
Expand Down
2 changes: 1 addition & 1 deletion scripts/pi-hole/php/FTL.php
Expand Up @@ -140,7 +140,7 @@ function disconnectFTL($quiet=true)
}

$clients = array();
function resolveHostname($clientip, $printIP)
function resolveHostname($clientip, $printIP = false)
{
global $clients;
$ipaddr = strtolower($clientip);
Expand Down

2 comments on commit 71a7c84

@pralor-bot
Copy link

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This commit has been mentioned on Pi-hole Userspace. There might be relevant details there:

https://discourse.pi-hole.net/t/clients-chart-not-loading-spinning-wheel/8362/12

@pralor-bot
Copy link

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This commit has been mentioned on Pi-hole Userspace. There might be relevant details there:

https://discourse.pi-hole.net/t/cant-see-my-graphs/8387/7

Please sign in to comment.