FHEM und pihole V6

Ralli · 05 März 2025, 16:11:19

Zitat von: Rudi_Hirsch am 01 März 2025, 14:46:04Ich habe keine Docker-Installation sondern eine raspi 3B-Installation

Zitatmein name: pihole -> { my $docker_jn=ReadingsVal('pihole','version_docker_local','unknown')} ->
leere Antwort.

$:-\$

Dann ist das das erwartete Verhalten.

juemuc · 05 März 2025, 17:14:56

Zitat von: Rudi_Hirsch am 05 März 2025, 15:53:59mein name: pihole -> { my $docker_jn=ReadingsVal('pihole','version_docker_local','unknown')} ->
leere Antwort.
Macht nix, funktioniert ja sonst alles

Gibt es denn bei Dir das Reading "version_docker_local"? Amsonsten verstehe ich das Verhalten nicht.

IM Bedarfsfall kannst Du ja die Abfrage in der 99_myUtils anpassen.

Viele Grüße
Jürgen

Ralli · 05 März 2025, 17:30:57

Schau gerne in die API-Dokumentation. Wird /info/version abgefragt, kommt ein JSON mit eben all diesen Werten zurück, inkl. Docker-Version(en) obwohl ggf. gar keine Docker-Image genutzt wird.

Ich verweise ein letztes mal auf meinen Beitrag, in dem ich die Docker-Versionen ausblende, wenn der Rückgabewert "" ist.

Rudi_Hirsch · 07 März 2025, 00:31:52

Gibt es denn bei Dir das Reading "version_docker_local"? Ansonsten verstehe ich das Verhalten nicht.
->
Alle Updateinfos sind aktuell vorhanden. Nur die beiden readings sind vorhanden aber leer

version_docker_local 2025-03-07 00:13:33
version_docker_remote 2025-03-07 00:13:33

juemuc · 07 März 2025, 12:56:45

Du hast aber schon - wie Ralli auch schon geschrieben hat - die Abfrage auf "leer" in Deinem Coding?

Code Auswählen

if($docker_jn eq 'unknown' or $docker_jn eq '') oder das Coding von Ralli.

Viele Grüße
Jürgen

Rudi_Hirsch · 08 März 2025, 16:08:38

Habe alles exakt nach RALLI gemacht (importiert). Nur bei 99_myUtils habe ich die Deaktivierungs-Zeiten nach Forum angepasst

Gisbert · 14 März 2025, 18:13:39

Hallo Jürgen,

ich hatte das Device und die Definition in 99_myUtils.pm geändert, teils nach meinen Vorstellungen, und alles sah sehr vernünftig und funktional aus.

Heute bekomme ich den im angehängten Bild gezeigten Zustand. Irgendetwas bei state scheint nicht mehr zu funktionieren, ohne dass ich mir irgendeiner Änderung bewusst bin.

Hier ist die Definition in 99_myUtils.pm:

Code Auswählen

sub piholeV6DevStateIcon
{
    my $name = shift;
    my $noHeader=shift;
    
    $noHeader=0 if(!defined($noHeader));
    
    #get %%base_url%% replacement from httpmod device (using HTTPMOD method)
    # -> this is the url to the web-ui of Pi-Hole
    my $baseURL= "https://".HTTPMOD::DoReplacement($defs{$name},'get','%%ip%%')."/admin/";

    #get required readings from pihole httpmod device
    my $state =ReadingsVal($name,'blocking','unknown');
    my $totalQueries = ReadingsVal($name,'queries_total','0');
    my $blocked = ReadingsVal($name,'queries_blocked','0');
    my $blockedPercentage = round(ReadingsVal($name,'queries_percent_blocked','0'),1);
    my $adListCount =ReadingsVal($name,'gravity_domains_being_blocked','0');
    my $messageCount=ReadingsNum($name,'count','0');
   
    my $ver_info = 'CORE: '.ReadingsVal($name,'version_core_local_version','unknown').' => '.ReadingsVal($name,'version_core_remote_version','unknown');
       $ver_info.= "\nFTL: ".ReadingsVal($name,'version_ftl_local_version','unknown').' => '.ReadingsVal($name,'version_ftl_remote_version','unknown');
       $ver_info.= "\nWEB: ".ReadingsVal($name,'version_web_local_version','unknown').' => '.ReadingsVal($name,'version_web_remote_version','unknown');
   
    my $ftl_upd=ReadingsNum($name,'FTL_update',0);
    my $core_upd=ReadingsNum($name,'core_update',0);
    my $web_upd=ReadingsNum($name,'web_update',0);

    my $anyUpd=$ftl_upd+$core_upd+$web_upd+0;
   

    #prepare dedicated css
    #hint: html of device overview is by default embedded in a <div> container
    #      which has id attribute set to device's NAME
    my $styleClass ='<style>';
       $styleClass.='#'.$name.' svg {height:15px; width:15px; padding-right:1px;} ';
       $styleClass.='#'.$name.' td {padding-left:10px; padding-right:10px} ';
       $styleClass.='</style>';
    
    #prepare images
    #ist schön bunt, wird aber nicht benötigt
    #my $imgQueries=FW_makeImage('it_i-net@cyan');
    #ist schön bunt, wird aber nicht benötigt    
    #my $imgBlocked=FW_makeImage('time_manual_mode@red');
    my $imgState  =FW_makeImage($state eq 'enabled' ? '15px-green' : '15px-red');
       $imgState  =FW_makeImage('15px-orange') if ($state eq 'enabled' and $messageCount > 0);
    my $imgUpdate =FW_makeImage($anyUpd > 0 ? '15px-red' : '15px-green',$ver_info);   

        if ($anyUpd > 0) {
                # fhem("setreading ".$name." available_updates 1")
                readingsSingleUpdate($defs{$name},"available_updates","1",1)
        } else {
                # fhem("setreading ".$name." available_updates 0")
                readingsSingleUpdate($defs{$name},"available_updates","0",0)
        }

    #prepare disable/enable command
    my $cmd='';
    #my $callMe='<a href="/fhem?cmd.'.$name.'=set '.$name.' PiHole';
    my $callMe='<a href="/fhem?cmd.'.$name.'=set '.$name . ' ';
    if($state eq 'disabled') {
        $cmd =$callMe.'enable">enable</a>';    
    } else {
        $cmd  =       $callMe.'disable 60">1</a>';
        $cmd .= ' | '.$callMe.'disable 300">5</a>';        
        $cmd .= ' | '.$callMe.'disable 600">10</a>';
       #$cmd .= ' | '.$callMe.'disable 900">15</a>';
    }

    #finally build the html for device overview
    my $ret  = $styleClass;
       $ret .= '<table border=0"><tr>';
       #if second argument of this method call is true (=1)
       #then do NOT create a table header. Defaults to false (=0)
       if(!$noHeader) {       
               $ret .= '<td><center>state</center></td>';
               $ret .= '<td><center>queries</center></td>';
               $ret .= '<td><center>blocked</center></td>';
               #$ret .= '<th>Adlist</th>';       
               $ret .= '<td><center>'.($state eq 'enabled' ? 'disabled':'disabled').'</center></td>';
            $ret .= '<td><center>update</center></td>';
               $ret .= '</tr><tr>';
       }       
       $ret .= '<td><center><a href="'.$baseURL.'?sid='.urlEncode(InternalVal("$name","sid","")).'" target="_blank">'.$imgState.'</a></center></td>';
       #$imgQueries und $imgBlocked entfernt
       #$ret .= '<td><center>'.$imgQueries.' '.$totalQueries.'</center></td>';
       #$ret .= '<td><center>'.$imgBlocked.' '.$blocked.' ('.$blockedPercentage.'%)'.'</center></td>';
       $ret .= '<td><center>'.$totalQueries.'</center></td>';
       $ret .= '<td><center>'.$blocked.' ('.$blockedPercentage.'%)'.'</center></td>';
       #$ret .= '<td>'.$imgAdList.' '.$adListCount.'</td>';
       $ret .= '<td><center>'.$cmd.'</center></td>';
       $ret .= '<td><center>'.$imgUpdate.'</center></td>';
       $ret .= '</tr></table>';

    return $ret;
}

Hab ich unwissentlich was geändert, oder woran könnte es liegen?

Viele Grüße Gisbert

bertl · 14 März 2025, 18:41:23

Hallo Gisbert,

das Bild/Icon "15px-orange.png" gibt es bei dir nicht (bei mir übrigens auch nicht).
Nimm einfach "15px-yellow" das ist vorhanden

Oder du kopierst dir unter "/opt/fhem/www/images/default" eines hin!

Gruß Robert

buzzmain · 17 April 2025, 17:39:48

Moin ihr Lieben,
für einen nicht so scriptversierten User (also z.B. Ich

) ist das echt kompliziert.
Wie aufwendig wäre es, PiHole als Funktion (nennt man das so?) ins Repository mit zu übernehmen.
Als so in der Art:

Code Auswählen

Define Pihole pihole IP-Adresse Passwort
Würde vielen vieles leichter machen.
Ist vermutlich zu aufwendig oder?

juemuc · 18 April 2025, 09:52:47

Was ist bei Copy& Paste so schwierig?

Ist doch nur kopieren und eigene Daten eintragen.

Jewe · 06 Juli 2025, 23:26:50

Hallo zusammen,

ich bekomme diesen fehler

Code Auswählen

error_hint The API is hosted at pi.hole/api, not pi.hole/admin/api
Was ist das Problem? Ich bekomme es nicht gefixed.

Danke, Jens

juemuc · 07 Juli 2025, 15:03:08

Hallo Jens,

die Info ist sehr spärlich.

Ich vermurte, Du nutz pihole v6 mit einer alten FHEM-Abfrage.

Viele Grüße
Jürgen

PS.: Ein paar zusätzliche Infos wären hilfreich (z. B. die Definition der "sub" und des Devices.

Jewe · 07 Juli 2025, 18:45:17

Hallo Jürgen,

ja die Info´s fehlen und haätte ich gleich einfügen können.
Die definitionen hab ich hier aus dem threat.

Device:

Code Auswählen

[code]define pihole HTTPMOD none 60
attr pihole DbLogExclude .*
attr pihole alias Pi-hole
attr pihole bodyDecode auto
attr pihole devStateIcon {piholeV6DevStateIcon($name)}
attr pihole enableControlSet 1
attr pihole enableCookies 1
attr pihole enforceGoodReadingNames 1
attr pihole event-on-change-reading .*
attr pihole extractAllJSON 1
attr pihole get01ExtractAllJSON 1
attr pihole get01Header X-FTL-SID: $sid
attr pihole get01Name FTL_summary
attr pihole get01Poll 1
attr pihole get01URL %%base_url%%/stats/summary
attr pihole get02ExtractAllJSON 1
attr pihole get02Header X-FTL-SID: $sid
attr pihole get02Name FTL_version
attr pihole get02Poll 1
attr pihole get02PollDelay 3600
attr pihole get02URL %%base_url%%/info/version
attr pihole get03ExtractAllJSON 1
attr pihole get03Header X-FTL-SID: $sid
attr pihole get03JSON status
attr pihole get03Name FTL_info
attr pihole get03Poll 1
attr pihole get03URL %%base_url%%/info/ftl
attr pihole get04Header X-FTL-SID: $sid
attr pihole get04JSON type
attr pihole get04Name FTL_status
attr pihole get04Poll 1
attr pihole get04URL %%base_url%%/dns/blocking
attr pihole get05Header X-FTL-SID: $sid
attr pihole get05JSON version
attr pihole get05Name FTL_messages
attr pihole get05Poll 1
attr pihole get05URL %%base_url%%/info/messages/count
attr pihole group System
attr pihole handleRedirects 1
attr pihole icon it_raspberry_logo
attr pihole reAuthRegex [Uu]nauthorized
attr pihole replacement01Mode text
attr pihole replacement01Regex %%base_url%%
attr pihole replacement01Value %%protocol%%://%%ip%%/api
attr pihole replacement02Mode text
attr pihole replacement02Regex %%ip%%
attr pihole replacement02Value DEINE_PIHOLE_IP_GGF_MIT_:PORT_WENN_NICHT_STANDARD
attr pihole replacement03Mode key
attr pihole replacement03Regex %%api_password%%
attr pihole replacement03Value api_password
attr pihole replacement04Mode text
attr pihole replacement04Regex %%protocol%%
attr pihole replacement04Value https
attr pihole replacement05Mode text
attr pihole replacement05Regex %%ip%%
attr pihole replacement05Value 192.168.6.2
attr pihole room Allgemein->Netzwerk,x_System
attr pihole set01Data {"blocking":false, "timer":$val}
attr pihole set01ExtractAllJSON 1
attr pihole set01Header X-FTL-SID: $sid
attr pihole set01Hint 10,60,180,300,600,900
attr pihole set01Max 900
attr pihole set01Min 10
attr pihole set01Name disable
attr pihole set01ParseResponse 1
attr pihole set01URL %%base_url%%/dns/blocking
attr pihole set02Data {"blocking":true}
attr pihole set02ExtractAllJSON 1
attr pihole set02Header X-FTL-SID: $sid
attr pihole set02Name enable
attr pihole set02NoArg 1
attr pihole set02ParseResponse 1
attr pihole set02URL %%base_url%%/dns/blocking
attr pihole sid1Data {"password": "%%api_password%%"}
attr pihole sid1IdJSON session_sid
attr pihole sidURL %%base_url%%/auth
attr pihole stateFormat blocking
attr pihole userReadings gravity_last_updated_abstime:gravity_last_update.* {strftime('%Y-%m-%d %H:%M:%S', localtime(ReadingsNum($name,'gravity_last_update',0)))}
#   BUSY       0
#   DEF        none 60
#   FUUID      68698e2b-f33f-3201-1771-56e9e9d1dc5cad32
#   FVERSION   98_HTTPMOD.pm:0.291590/2024-09-23
#   Interval   60
#   MainURL    
#   ModuleVersion 4.2.0 - 11.8.2023
#   NAME       pihole
#   NOTIFYDEV  global
#   NR         816
#   NTFY_ORDER 50-pihole
#   STATE      blocking
#   TYPE       HTTPMOD
#   value      
#   CompiledRegexes:
#   HttpUtils:
#     data       
#     displayurl https://DEINE_PIHOLE_IP_GGF_MIT_:PORT_WENN_NICHT_STANDARD/api/info/messages/count
#     header     X-FTL-SID: $sid
#     httpversion 1.0
#     hu_blocking 0
#     ignoreredirects 1
#     loglevel   4
#     redirects  0
#     timeout    2
#     url        https://DEINE_PIHOLE_IP_GGF_MIT_:PORT_WENN_NICHT_STANDARD/api/info/messages/count
#     sslargs:
#   QUEUE:
#   READINGS:
#     2025-07-07 18:39:44   available_updates 0
#     2025-07-05 22:42:41   error_hint      The API is hosted at pi.hole/api, not pi.hole/admin/api
#     2025-07-05 22:42:41   error_key       bad_request
#     2025-07-05 22:42:41   error_message   Bad request
#     2025-07-05 22:42:41   took            6.53266906738281e-05
#   REQUEST:
#     context    get
#     data       
#     header     X-FTL-SID: $sid
#     ignoreredirects 0
#     num        05
#     retryCount 0
#     type       get05
#     url        %%base_url%%/info/messages/count
#   lastpoll:
#     FTL_info   1751906572.2516
#     FTL_messages 1751906572.2516
#     FTL_status 1751906572.2516
#     FTL_summary 1751906572.2516
#     FTL_version 1751906332.17786
#
setstate pihole blocking
setstate pihole 2025-07-07 18:39:44 available_updates 0
setstate pihole 2025-07-05 22:42:41 error_hint The API is hosted at pi.hole/api, not pi.hole/admin/api
setstate pihole 2025-07-05 22:42:41 error_key bad_request
setstate pihole 2025-07-05 22:42:41 error_message Bad request
setstate pihole 2025-07-05 22:42:41 took 6.53266906738281e-05

[/code]

in der 99_myutils:

Code Auswählen

######################################################################################
#### PiHole piholeV6DevStateIcon
#
sub piholeV6DevStateIcon
{
    my $name = shift;
    my $noHeader=shift;
   
    $noHeader=0 if(!defined($noHeader));
   
    #get %%base_url%% replacement from httpmod device (using HTTPMOD method)
    # -> this is the url to the web-ui of Pi-Hole
    my $baseURL= "https://".HTTPMOD::DoReplacement($defs{$name},'get','%%ip%%')."/admin/";

    #get required readings from pihole httpmod device
    my $state =ReadingsVal($name,'blocking','unknown');
    my $totalQueries = ReadingsVal($name,'queries_total','0');
    my $blocked = ReadingsVal($name,'queries_blocked','0');
    my $blockedPercentage = round(ReadingsVal($name,'queries_percent_blocked','0'),1);
    my $adListCount =ReadingsVal($name,'gravity_domains_being_blocked','0');
    my $messageCount=ReadingsNum($name,'count','0');
  
    my $ver_info = 'CORE: '.ReadingsVal($name,'version_core_local_version','unknown').' => '.ReadingsVal($name,'version_core_remote_version','unknown');
       $ver_info.= "\nFTL: ".ReadingsVal($name,'version_ftl_local_version','unknown').' => '.ReadingsVal($name,'version_ftl_remote_version','unknown');
       $ver_info.= "\nWEB: ".ReadingsVal($name,'version_web_local_version','unknown').' => '.ReadingsVal($name,'version_web_remote_version','unknown');
  
    my $ftl_upd=ReadingsNum($name,'FTL_update',0);
    my $core_upd=ReadingsNum($name,'core_update',0);
    my $web_upd=ReadingsNum($name,'web_update',0);

    my $anyUpd=$ftl_upd+$core_upd+$web_upd+0;
  

    #prepare dedicated css
    #hint: html of device overview is by default embedded in a <div> container
    #      which has id attribute set to device's NAME
    my $styleClass ='<style>';
       $styleClass.='#'.$name.' svg {height:15px; width:15px; padding-right:1px;} ';
       $styleClass.='#'.$name.' td {padding-left:10px; padding-right:10px} ';
       $styleClass.='</style>';
   
    #prepare images
    #ist schön bunt, wird aber nicht benötigt
    #my $imgQueries=FW_makeImage('it_i-net@cyan');
    #ist schön bunt, wird aber nicht benötigt   
    #my $imgBlocked=FW_makeImage('time_manual_mode@red');
    my $imgState  =FW_makeImage($state eq 'enabled' ? '15px-green' : '15px-red');
       $imgState  =FW_makeImage('15px-orange') if ($state eq 'enabled' and $messageCount > 0);
    my $imgUpdate =FW_makeImage($anyUpd > 0 ? '15px-red' : '15px-green',$ver_info);  

        if ($anyUpd > 0) {
                # fhem("setreading ".$name." available_updates 1")
                readingsSingleUpdate($defs{$name},"available_updates","1",1)
        } else {
                # fhem("setreading ".$name." available_updates 0")
                readingsSingleUpdate($defs{$name},"available_updates","0",0)
        }

    #prepare disable/enable command
    my $cmd='';
    #my $callMe='<a href="/fhem?cmd.'.$name.'=set '.$name.' PiHole';
    my $callMe='<a href="/fhem?cmd.'.$name.'=set '.$name . ' ';
    if($state eq 'disabled') {
        $cmd =$callMe.'enable">enable</a>';   
    } else {
        $cmd  =       $callMe.'disable 60">1</a>';
        $cmd .= ' | '.$callMe.'disable 300">5</a>';       
        $cmd .= ' | '.$callMe.'disable 600">10</a>';
       #$cmd .= ' | '.$callMe.'disable 900">15</a>';
    }

    #finally build the html for device overview
    my $ret  = $styleClass;
       $ret .= '<table border=0"><tr>';
       #if second argument of this method call is true (=1)
       #then do NOT create a table header. Defaults to false (=0)
       if(!$noHeader) {      
               $ret .= '<td><center>state</center></td>';
               $ret .= '<td><center>queries</center></td>';
               $ret .= '<td><center>blocked</center></td>';
               #$ret .= '<th>Adlist</th>';      
               $ret .= '<td><center>'.($state eq 'enabled' ? 'disabled':'disabled').'</center></td>';
            $ret .= '<td><center>update</center></td>';
               $ret .= '</tr><tr>';
       }      
       $ret .= '<td><center><a href="'.$baseURL.'?sid='.urlEncode(InternalVal("$name","sid","")).'" target="_blank">'.$imgState.'</a></center></td>';
       #$imgQueries und $imgBlocked entfernt
       #$ret .= '<td><center>'.$imgQueries.' '.$totalQueries.'</center></td>';
       #$ret .= '<td><center>'.$imgBlocked.' '.$blocked.' ('.$blockedPercentage.'%)'.'</center></td>';
       $ret .= '<td><center>'.$totalQueries.'</center></td>';
       $ret .= '<td><center>'.$blocked.' ('.$blockedPercentage.'%)'.'</center></td>';
       #$ret .= '<td>'.$imgAdList.' '.$adListCount.'</td>';
       $ret .= '<td><center>'.$cmd.'</center></td>';
       $ret .= '<td><center>'.$imgUpdate.'</center></td>';
       $ret .= '</tr></table>';

    return $ret;
}

Ralli · 07 Juli 2025, 21:06:37

Zitat von: Jewe am 07 Juli 2025, 18:45:17
Code Auswählen Erweitern
attr pihole replacement02Value DEINE_PIHOLE_IP_GGF_MIT_:PORT_WENN_NICHT_STANDARD

Dein Ernst?

Jewe · 08 Juli 2025, 19:00:14

Dein Ernst?
[/quote]

Hallo Ralli,

jaein, ich hatte auch noch den replacement05Value mit der IP drin und das wohl übersehen. Mein Fehler und auch dass ich die Vorlage auf das alte Define übernommen hatte war auch nicht so wirklich klug. Hatte gedacht ich mache es mir einfach, stattdessen viel Arbeit auch für Euch. Sorry.

Habe das gerät nun gelöscht und komplett neu gemacht und so funkttioniert es auch.

LG, Jens