SNMP Modul

tommes · 17. November 2023 um 19:11

Hallo zusammen,

super geniales Modul. Vielen Dank dafür.
Die Zusammenarbeit mit dem Konverter von mib in oidlibs funktioniert gut, so dass man sich auch Teile des Baumes anzeigen kann.

Aber:

Eine kleine Korrektur müsst Ihr meines Erachtens noch machen:
Das Häkchensetzen erstellt sofort eine Variable und das häkchen entfernen löscht sie sofort.
Das widerspricht Euren eigenen Modulrules und - ganz wichtig - es führt zu Datenverlust wenn man mal eine Variable achiviert und dann versehentlich auf das Häkchen kommt.
Weiterhin hat man ein Problem wenn die Variable verknüpft oder anderweitig verwendet wird, dann ist alles pfutsch.

Vielleicht könnte man irgendwo ein Häkchen setzen, dass Variablen nicht gelöscht werden. Es würde ja reichen ein Präfix („to_delete“) vor den Varnamen zu setzen, gelöscht ist es dann manuell ganz schnell.

Beste Grüße
Tom

Nall-chan · 17. November 2023 um 19:21

Nö, dazu wird hier nix geschrieben:

gist.github.com

https://gist.github.com/paresy/236bfbfcb26e6936eaae919b3cfdfc4f

gistfile1.md

## Best Practice zur PHP-Modul Erstellung

1. **Generelle Entwicklung**
   1. Module sollten grundsätzlich auf Englisch erstellt und ins Deutsche übersetzt werden
   2. Fehlermeldungen dürfen mit @ nur verdeckt werden, wenn der Rückgabewert überprüft wird und bei einem echten Fehlerfall dies dem Nutzer mitgeteilt wird. Andernfalls findet man im Fehlerfall den Verursacher niemals heraus. Sichtbare Fehler können behoben werden. Unsichtbare Fehler bringen inkonsistentes Verhalten und frustrieren nur alle beteiligten.
   3. Daten, welche der Benutzer nicht benötigt, sollten auch nicht in Variablen erscheinen (z.B. irgendwelche Puffer / Temporäre Inhalte in String Variablen). Dafür bietet das SDK die SetBuffer/GetBuffer Funktionen, welches über eine JSON Kodierung auch mehrere Elemente enthalten können.
   4. Der Inhalt der library.json / module.json darf nur die von der Dokumentation definierten Felder enthalten. Andere Felder können in zukünftigen Version seitens IP-Symcon verwendet werden und sind vollständig reserviert.
   5. Module mit externen Abhängigkeiten müssen diese vollständig im speziell erlaubten libs Ordner mitliefern.
   6. Ein Modul darf niemals Abhängigkeiten aus einer anderen Bibliothek erfordern, sodass das Modul (ohne diese Bibliothek) "kaputt" ist (z.B. ein include auf eine benötigte PHP Datei). Eine Bibliothek muss immer in sich geschlossen und funktionsfähig sein. Optionale Funktionen können durch zusätzlich installierte Bibliotheken freigeschaltet werden. (Randfall: Ein Modul *kann* ohne sinvolle Funktion sein, solange keine anderen installierten Bibliotheken vorhanden sind. Sie muss aber darauf Hinweisen und trotzdem fehlerfrei installierbar sein. Sobald weitere unterstützte Bibliothken installiert werden, erweitert sich der Funktionsumfang)
   7. Objekte dürfen niemals über den Namen gefunden werden. Es *muss* immer ein Ident gesetzt und dafür verwendet werden.

This file has been truncated. show original

Aber vielleicht wäre eine Warnmeldung vorher ganz nett
Michael

paresy · 17. November 2023 um 19:22

Wir können ja mal schauen, ob da eine Warnmeldung oder Bestätigung einbaubar ist.

paresy

tommes · 17. November 2023 um 19:56

Ok ihr habt recht, ich hatte das mit dem Punkt „Eine Instanz sollte niemals automatisch Variablen im Archiv aktivieren“ falsch in Erinnerung.

Das ist zwar jetzt offtopic, aber Variablen, die man Archiviert oder die mit anderen Aktionen verknüpft sind, dürften auf keinen Fall gelöscht werden. Unter Umständen findet man einen verschwundenen Trigger erst nach Wochen, weil man sich wundert, dass gar keine Meldungen mehr kommen.

Die Idee mit der Warnmeldung finde ich sehr gut, wenn sie abbrechbar ist und nur erscheint, wenn die Variable archiviert oder referenziert ist.

Grüße Tom

tommes · 20. November 2023 um 17:50

Hallo zusammen,
ich habe noch eine Frage zum Modul: ich habe bei manchen Geräten einen Abbruch des Walkes mit einer Fehlermeldung:

A non-numeric value encountered in /var/lib/symcon/modules/.store/de.symcon.snmp/libs/vendor/freedsx/asn1/src/FreeDSx/Asn1/Encoder/BerEncoder.php on line 550

Warning: A non-numeric value encountered in /var/lib/symcon/modules/.store/de.symcon.snmp/libs/vendor/freedsx/asn1/src/FreeDSx/Asn1/Encoder/BerEncoder.php on line 559

Warning: A non-numeric value encountered in /var/lib/symcon/modules/.store/de.symcon.snmp/libs/vendor/freedsx/asn1/src/FreeDSx/Asn1/Encoder/BerEncoder.php on line 560
The requested OID (1.3.6.1.4.1.5040.1.1.1.3.4.1.12.1) cannot be returned (NoSuchName).

Nach dieser Meldung bricht der Walk komplett ab und die Liste ist leer, obwohl schon 80 OID’s gefunden wurden. Könnte man einen solchen Wert einfach überspringen? So kommt man an gar keine Werte heran.

Viele Grüße
Tom

tobiasr · 26. November 2023 um 21:41

Da ich davon ausgehe, dass Acer90 sein Modul mittelfristig nicht weiter pflegen wird oder es sogar schon aus dem Store verschwunden ist:
Besteht die Möglichkeit, auch mittels des offiziellen Moduls nur Nutzer, Passwort, Host zu speichern und weiterhin über Scripte die Abfragen zu triggern?