[Release] script.service.caretaker

PvD · 17. August 2017, 16:04

Diskussionen:
-> Diskussionsthema rund um die Möglichkeiten des Caretaker-Addons

Aus der addon.xml:

Quellcode

Dieser Dienst überprüft den Addon-Ordner auf nicht autorisierte, möglicherweise schädliche Repositories.

Das macht er anhand einer Blacklist, die hier Forum zur Verfügung gestellt wurde. Wird ein geblacklistetes (was für ein Wort) Repo gefunden, erfolgt eine Notification. Möchte man wissen, welches Repo das ist, muss man vorerst das 'caretaker.log' bemühen.

Es erfolgt kein IP-Tracking, das übernehmen evtl. die installierten illegalen Adons! Ebenso wird kein Systemscan vorgenommen, es erfolgt lediglich eine Abfrage der Addon-Datenbank von Kodi!

Das Addon arbeitet als Service, ist also skinunabhängig. Daneben gibt es noch die Möglichkeit, über Programme -> KN Addon Caretaker eine Analyse des Addon-Verzeichnisses zu starten. Dabei wird ein Log namens 'caretaker.log' erstellt, welches im gleichen Verzeichnis wie das kodi.log liegt.

PvD · 17. August 2017, 20:30

War noch ein Bugfix fällig. Neue Version im Post #1.

L0RE · 7. September 2017, 16:34

Noch nciht genauer überlegt wäre es möglich es so zu bauen aus anderen Plugins Möglich ist:

import caretaker

if caretaker.check() ==1:
# Notify Dieses Plugin läuft nur auf Sauberen Systemen
exit

PvD · 7. September 2017, 17:30

In den Settings ist eine Option 'permanente Überwachung ' per Default on. Damit wird alle 30 min ein Check angestossen und das Addon bringt dann natürlich alle 30 Min. eine Notification, sofern ein BL Repo erkannt wird. Und die meisten werden nicht wissen, wie sich das abstellen lässt.

Was hältst Du von einer Abhängigkeit bei der Installation des Kodinerds-Repos? Der Caretaker würde dann mitinstalliert werden...

L0RE · 7. September 2017, 18:25

Das ist schwierig. Wenn einer Kodinerds noch nicht hat, kann es über die Abhängigkeiten nicht aufgelöst werden

Das mit den include such ich mal raus....

PvD · 7. September 2017, 18:35

Naja, ich möchte nicht zu tief in die Privatsphäre eines Nutzers eingreifen. Wenn derjenige nicht mal mehr Addons installieren kann, das geht dann doch zu weit.

L0RE · 7. September 2017, 19:30

Naja das wäre ja die Entscheidung jedes Entwicklers ob er es nutze will. Ich wüsste es auch noch nicht ob ich es nutze. Hatte aber schon 3-4 User anfragen bei denen Mein Addon nur wegen den Illegalen schei*** nicht lief... . fInd aber immer gut wenn es so eine Funktionalität gibt . Wobei die Liste Fehler hat, hab es BJ1 geschrieben

don · 8. September 2017, 10:05

Das wäre vielleicht eher auf Ebene von LibreELEC zu machen?!
Man installiert sich Libre und bekommt den Hinweis, wenn man sich unwissentlich eine fiese Repo installiert.
Wobei ... ginge auch auf nerdsrepo-Ebene. Die Repo ist ja bei Libre schon mit enthalten.
Es soll sich ja jeder sein System versauen dürfen, wie er will.
Aber ein Hinweis ist eine wirklich gute Idee. Hatten ja schon ausreichend Fälle, bei denen der unbedarfte Schreiber nicht wußte, was er da installiert hat.

L0RE · 8. September 2017, 11:35

Hab mal @BJ1 ne Version geschickt.
Da hätte man ne Funktion die man aufrufen kann aus nem Addon und er gibt 1 oder 0 zurück dann kan man entscheiden was man macht
Hab mir überlegt wie mans machen kann....

Würde ich warscheinlich in allen Addons einbaun das er das als Debug Info Mit ausgibt , vileicht mit nem Hashwerd, das es nicht ganz offensichtlich ist. So das ich beim Supporten schneller enscheiden kann ob ich nach suchen muss, oder obs vieleicht an seiner verhunzten Instalation liegt. Nerft wenn man ewig debuggt und dan feststellt der hat sich jeden ****** instaliert.... Vieleicht noch nen Warungs Text in den Settings...

PvD · 26. Oktober 2017, 19:00

Horstepipe schrieb:

Mal nebenbei gefragt, ist das TVTunes-Addon Teil der Blacklist? Bin da nicht mehr auf dem Stand der Dinge, wie es darum steht. Würde es nur demnächst gerne wieder nutzen.

Der Caretaker schaut nach blacklisted Repos und gibt eine Hinweismeldung/Warnmeldung aus, sofern er ein Repo findet - das allerdings alle 30 Minuten. Mehr macht er nicht. Daneben kann man sich mit dem Start über Addons -> Programme usw... ein detailliertes Log generieren lassen, welches im selben Verzeichnis wie das kodi.log liegt.

Der Benefit ist - wie sualfred schon angemerkt hat - dass ein User auch informiert wird, wenn er - bspw. unbedarft - ein banned Repo installiert hat. Kaputt geht nix und auch Addons lassen sich ja konfigurieren

. Allerdings wissen das die wenigsten, die sich per Wizard oder über 'full loaded' Boxen bedienen...

PvD · 27. Oktober 2017, 22:01

tehax schrieb:

Nebenbei bemerkt bin ich gegen Malware denn das ist dieses Script.

Dieses Script ist weder Malware, Trojaner, Virus noch ein Backdoor-Programm. Im Gegenteil, es ist vollkommen harmlos. Ich muss es wissen.
de.wikipedia.org/wiki/Schadprogramm

sualfred · 29. Oktober 2017, 13:24

@BJ1

Quellcode

13:22:37.958 T:14420 ERROR: EXCEPTION Thrown (PythonToCppException) : -->Python callback/script returned the following error<--
- NOTE: IGNORING THIS CAN LEAD TO MEMORY LEAKS!
Error Type: <type 'exceptions.WindowsError'>
Error Contents: [Error 123] Die Syntax f?r den Dateinamen, Verzeichnisnamen oder die Datentr?erbezeichnung ist falsch: 'special:'
Traceback (most recent call last):
File "C:\Users\sualfred\AppData\Roaming\Kodi\addons\script.service.caretaker\service.py", line 47, in <module>
run_service()
File "C:\Users\sualfred\AppData\Roaming\Kodi\addons\script.service.caretaker\service.py", line 6, in run_service
updateBlacklist(BLACKLIST_CACHE, BLACKLIST_REMOTE, BLACKLIST)
File "C:\Users\sualfred\AppData\Roaming\Kodi\addons\script.service.caretaker\resources\lib\tools.py", line 79, in updateBlacklist
os.makedirs(ADDON_PROFILE)
File "D:\Kodi\Kodi Krypton\system\python\Lib\os.py", line 150, in makedirs
makedirs(head, mode)
File "D:\Kodi\Kodi Krypton\system\python\Lib\os.py", line 150, in makedirs
makedirs(head, mode)
File "D:\Kodi\Kodi Krypton\system\python\Lib\os.py", line 150, in makedirs
makedirs(head, mode)
File "D:\Kodi\Kodi Krypton\system\python\Lib\os.py", line 157, in makedirs
mkdir(name, mode)
WindowsError: [Error 123] Die Syntax f?r den Dateinamen, Verzeichnisnamen oder die Datentr?erbezeichnung ist falsch: 'special:'
-->End of Python script error report<--

Alles anzeigen

Seit v7.0

PvD · 29. Oktober 2017, 13:52

Nein, schon immer. Ist nur niemanden aufgefallen, da der Umstand nicht eingetreten ist, dass das Setup-Verzeichnis im userdata noch nicht angelegt war.
Wie dem auch sei, v0.0.8 ist im ersten Post.

sualfred · 29. Oktober 2017, 13:59

BJ1 schrieb:

Nein, schon immer. Ist nur niemanden aufgefallen, da der Umstand nicht eingetreten ist, dass das Setup-Verzeichnis im userdata noch nicht angelegt war.
Wie dem auch sei, v0.0.8 ist im ersten Post.

0.0.8 ist aber 0.0.7 und aufm GitHub ist auch nichts passiert

PvD · 29. Oktober 2017, 14:26

Ja vergessen, die addon.xml zu aktualisieren. Github lade ich gleich hoch.

PvD · 30. Oktober 2017, 18:58

War mir klar. Siehe Punkt 1 und 3 dieses Posts hier.

[Release] script.service.caretaker

Der Caretaker (und die ihn verwenden) wird banned, weil er banned Addons aufzeigt. Welche Ironie und Scheinheiligkeit.

.

Saxnot · 30. Oktober 2017, 21:22

Sorry, das ich in die spannende Debate mit ner dummen Frage unterbreche

Bei mir gibt der Caretaker folgendes aus:

Quellcode

################################################################################
# ... output of cat /storage/.kodi/temp/caretaker.log
# LibreELEC release: Generic.x86_64-devel-20171023210317-#1023-gff23ff4
#
################################################################################
*** Starting KN Addon Caretaker V.0.0.7 at 2017-10-30 20:43 ***
Use blacklist with timestamp 2017-10-27 18:18
430 blacklisted items loaded
Check addon folder for blacklisted repositories
Repository 'repository.zachmorris' found
installed from repo: 'plugin.program.iarl' (2017-10-07 21:27)

Alles anzeigen

Ist der IARL mit Absicht auf der Liste? Okay, die Frage kann ich mir selbst beantworten, da es eine Blacklist ist, ist er mit Absicht drauf :rolleyes:

. Aber ist das nicht etwas zu restriktiv, wo doch das Addon von Kodi.tv selbst verlinkt wird?

PvD · 30. Oktober 2017, 21:54

@Saxnot: zachmorris steht auf der Blacklist. Die Info, dass dieses Repo über iarl installiert wurde, kommt aus der addons.db. Das heisst nicht, dass iarl auf der Blacklist ist.

Saxnot · 30. Oktober 2017, 22:17

BJ1 schrieb:

@Saxnot: zachmorris steht auf der Blacklist. Die Info, dass dieses Repo über iarl installiert wurde, kommt aus der addons.db. Das heisst nicht, dass iarl auf der Blacklist ist.

Ah, verstehe, danke!
Im zachmorris-Repo ist nur der iarl drin, das blacklisting der repo müsste dann also an dem addon iarl liegen, oder?
Vll. wäre es gut, in dem Startposting noch ein paar Worte dazu zu verlieren, nach welchen Kriterien die Blacklist zu stande kommt!?

PvD · 30. Oktober 2017, 22:39

Saxnot schrieb:

Ah, verstehe, danke!
Im zachmorris-Repo ist nur der iarl drin, das blacklisting der repo müsste dann also an dem addon iarl liegen, oder?
Vll. wäre es gut, in dem Startposting noch ein paar Worte dazu zu verlieren, nach welchen Kriterien die Blacklist zu stande kommt!?

Ich lege das gerne offen (btw. ist der Quelltext ja ohnehin verfügbar). Entsprechende Änderungen sind auch per PR gern gesehen. Endlich mal ein sachlicher und konstruktiver Beitrag. :thumbup:

Quellcode

Quellcode

Quellcode

Ähnliche Themen