IPSLogger - mein kleiner feiner LogHandler für IPS

Erweiterungen IPSLibrary
281

Hallo Andreas,

ich hätte einen Wunsch zum Logging Format ‚Log4IPS‘, das ich sehr intensiv nutze.

Es wäre schön, wenn du die Ausgabe um die ThreadID erweitern könntest. Die ThreadID ist zum Filtern der Logzeilen oft sehr hilfreich.

Ich habe bei mir lokal die folgende Funktion erweitert:

	// ---------------------------------------------------------------------------------------------------------------------------
	function IPSLogger_OutLog4IPS($LogLevel, $LogType, $Context, $Msg) {
			$Out  = '<event';
			$Out .=   ' logger="'.$Context.'"';
			$Out .=   ' timestamp="'.date('Y-m-d\TH:i:s.u').'+01:00"';
			$Out .=   ' level="'.IPSLogger_LogTypeXml($LogType).'"';
			$Out .=   ' domain="IPS.exe"';
			$Out .=   ' thread="'.$_IPS['THREAD'].'"'; // meine Ergänzung
			$Out .=   ' username="IPS">';
			$Out .=   '<message>'.$Msg.'</message>';
			$Out .= '</event>';

			$File = 'IPSLogger_'.date('Ymd').'.'.c_Log4IPS_Extension;
			IPSLogger_WriteFile(c_Log4IPS_Directory, $File, $Out, c_ID_Log4IPSOutEnabled);
		}

Das wäre super :slight_smile:

Gruß

Burkhard

282

Hallo Andreas,

ab und zu (so alle ein bis zwei Wochen) kommt es vor, dass es beim Loggen zu einer Fehlermeldung kommt und das Logging anschließend abgeschaltet ist:

(7) Warning: fopen(C:\IP-Symcon\logs\IPSLogger_20170105.xml): failed to open stream: Permission denied
   Error in Script C:\IP-Symcon\scripts\IPSLibrary\app\core\IPSLogger\IPSLogger_Output.inc.php on Line 53
  134 in scripts\IPSLibrary\app\core\IPSLogger\IPSLogger.inc.php (call IPSLogger_Out)
   39 in scripts\IPSLibrary\app\core\IPSLogger\IPSLogger_PhpErrorHandler.inc.php (call IPSLogger_Err)
      in IPSLogger_PhpErrorHandler
   53 in scripts\IPSLibrary\app\core\IPSLogger\IPSLogger_Output.inc.php (call fopen)
  101 in scripts\IPSLibrary\app\core\IPSLogger\IPSLogger_Output.inc.php (call IPSLogger_WriteFile)
      in IPSLogger_OutLog4IPS
  506 in scripts\IPSLibrary\app\core\IPSLogger\IPSLogger_Output.inc.php (call call_user_func_array)
   39 in scripts\IPSLibrary\app\core\IPSLogger\IPSLogger_Output.inc.php (call IPSLogger_invokeLoggers)
  205 in scripts\IPSLibrary\app\core\IPSLogger\IPSLogger.inc.php (call IPSLogger_Out)
   71 in scripts\WOL.ips.php (call IPSLogger_Trc)
   31 in scripts\WOL.ips.php (call online)

Ich vermute, dass in dieser Situation zwei Threads gleichzeitig schreiben wollen.

Ich habe die Funktion ‚IPSLogger_WriteFile‘ daher bei mir um eine Semaphore erweitert:


function IPSLogger_WriteFile($Directory, $File, $Text, $ID_OutEnabled) {
	if ($Directory == "") {
		$Directory = IPS_GetKernelDir().'logs/';
		if (function_exists('IPS_GetLogDir'))
			$Directory = IPS_GetLogDir();
	}
      //Semaphor eingebaut um einen zeitgleichen Zugriff ("Warning: fopen ...: failed to open stream: permission denied")
      //zu verhindern.
      //Da im Fall eines 'Parse Errors' keine Semaphore erhältlich ist ('Warning:  Cannot detect ThreadID!')
      //sollten die Warnings unterdrückt werden (@)
      @IPS_SemaphoreEnter(__FUNCTION__.$Directory.$File, 1000);

      if(($FileHandle = fopen($Directory.$File, "a")) === false) {
         SetValue($ID_OutEnabled, false);
         @IPS_SemaphoreLeave(__FUNCTION__.$Directory.$File);
         return;
      }
      fwrite($FileHandle, $Text.c_lf);
      fclose($FileHandle);
      @IPS_SemaphoreLeave(__FUNCTION__.$Directory.$File);
}

Seitdem ist Ruhe:)

Wäre schön, wenn du die Änderungen übernehmen könntest.

Gruß

Burkhard

283

OK, sieht brauchbar aus :slight_smile:

Hab mir das mal notiert und werde es beim nächsten Update einpflegen …

284

Besten Dank im Voraus. Dann bin ich wieder synchron:)

285

Hallo Andreas,

ich nutze wegen der guten Filtermöglichkeiten gerne ‚Log4IPS‘ in Verbindung mit Log4View. Dabei sind mir zwei Fehler aufgefallen, die ich in der Funktion IPSLogger_OutLog4IPS behoben habe:


	// ---------------------------------------------------------------------------------------------------------------------------
	function IPSLogger_OutLog4IPS($LogLevel, $LogType, $Context, $Msg) {
         // bumaas: Sonderzeichen entfernen um Lesefehler von Log4View zu vermeiden
         $Msg  =  str_replace (array('<','>'), '.', $Msg); 
			$Out  = '<event';
			$Out .=   ' logger="'.$Context.'"';
			$Out .=   ' timestamp="'.date('Y-m-d\TH:i:s.u').'+01:00"';
			$Out .=   ' level="'.IPSLogger_LogTypeXml($LogType).'"';
			$Out .=   ' domain="IPS.exe"';
			$Out .=   ' thread="'.$_IPS['THREAD'].'"';
			$Out .=   ' username="IPS">';
         // bumaas: die Ausgabe muss utf8 decodiert werden
			$Out .=   '<message>'.utf8_decode($Msg).'</message>';
			$Out .= '</event>';

			$File = 'IPSLogger_'.date('Ymd').'.'.c_Log4IPS_Extension;
			IPSLogger_WriteFile(c_Log4IPS_Directory, $File, $Out, c_ID_Log4IPSOutEnabled);
		}

Es wäre schön, wenn du die Korrekturen ebenfalls in den nächsten Stand einfließen lassen könntest.

Gruß

Burkhard

286

Hab die Änderungen übernommen, Update ist jetzt online …

287

Besten Dank!

288

Hallo,

gibt es eine Möglichkeit bestimmte Scripte vom Logging auszuschließen. Ich habe ein Script das ich gerne nicht überwacht hätte.

289

Meinst du Fehlermeldungen und Warnings? Die kannst du mit ‚@‘ bzw. einem Exception Handling (‚try‘) unterdrücken.

290

Habe leider noch eine Anweisung übersehen:


// bumaas: die Ausgabe muss utf8 decodiert werden
$Out .=   ' logger="'.utf8_decode($Context).'"';

Vielleicht kannst du das bei Gelegenheit auch noch glattziehen.

LG Burkhard