Schrolli91 / BOSWatch

Python Script to process input data from rtl_fm and multimon-NG - multiple Plugin support
https://bwcc.boswatch.de
GNU General Public License v2.0
132 stars 59 forks source link

Email-Fehler im Master-Branch #197

Closed Thiwol closed 7 years ago

Thiwol commented 7 years ago

Hallo,

im Master-Branch sind in der eMail.py zwei Fehler. Es wurde vergessen, den Variablennamen "globals" in "globalVars" zu ändern.

flothi commented 7 years ago

Ist mit pull-request erledigt.

Schrolli91 commented 7 years ago

Dürfte ja dann damit behoben sein ...

Thiwol commented 7 years ago

Jup, hatte noch einen anderen Fehler. Ich wollte die Email mit SSL versenden, was zuerst nicht geklappt hat, da immer die starttls-Funktion aufgerufen worden ist. Habe die Zeile einfach auskommentiert, kann es sein, dass die IF-Abfrage nicht funktioniert?

flothi commented 7 years ago

Das Problem hab ich auch, da einer meiner Server offensichtlich auf dem SSL-Ohr taub ist Ich hatte nur wenig Zeit, meinen Fix zu testen. Ggf müsste die Exception einfach nur gefangen werden.

Mal sehen wie mein Abend läuft ;-)

flothi commented 7 years ago

Ich habe den Fix in den Master gepullt ( #200 )- zum Einen ist das Plugin mMn recht wichtig, zum Anderen war die Abfrage in der Tat fehlerhaft (get anstatt getboolean). Weiterhin habe ich das SMTP_SSL mit einem weiteren try-except-Block versehen; einige meiner Mailserver sprechen (warum auch immer) nicht mit der SSL-Bib von python, obwohl sie es eigentlich können sollten.

flothi commented 7 years ago

@Thiwol Ist der Fehler beseitigt?

Thiwol commented 7 years ago

@flothi Jap, es funktioniert.

flothi commented 7 years ago

Danke fürs Feedback!

chbally commented 7 years ago

Sorry - ich glaube der Fehler existiert in abgewandelter Form noch. Ich kann mit meinem Mailserver nicht per SSL kommunizieren. Ich habe trotz massiver versuche keine Mail mit SMTP über Port 25 ohne TLS zustellen können. Gleiches berichtet ein Kollege. Per SSMTP mit einem anderen Server, den ich nicht eigentlich verwenden möchte, geht es einwandfrei.

flothi commented 7 years ago

Welchen Server verwendest du denn? Magst du bitte mal in deiner plugins/eMail/eMail.py in Zeile 111 den Wert für Debug auf 1 schalten und das Ergebnis mitteilen. (und boswatch mit der Option -v starten)

chbally commented 7 years ago

Das kann ich morgen gerne mal machen. Das ist ein Standardmailserver eines Providers, über den ich normal über SMTP Port 25 kommuniziere. Die config.ini sieht so aus:

[eMail]
# SMTP-Server
smtp_server = mail.meine-domain.de
# Port of SMTP-Server (default:
smtp_port = 25
# use tls for connection (0|1)
tls = 0
# Use this, when SMTP-Server has restricted access
user = alarm@meine-domain.de
password = eingeheimesPasswort

# Parameters for Alarm-Msg:
# "to" could be more than one address, comma separated
from = alarm@meine-domain.de
to   = user@irgendwo.de

# Priority of the eMail:
# normal|urgent|non-urgent
priority = urgent

Mit einem anderen Skript funktioniert das mit den Daten vom gleichen Raspberry.

chbally commented 7 years ago

Fehlalarm: Ich habe meine komplette config.ini umgebaut und nun geht es. Sorry dafür!!!

Kleine Frage am Rande: Kann man beim Feld from = alarm@meine-domain.de noch einen Klarnamen als Absender wie ALARMSYSTEM FF XY einfügen

By the way: Tolle Arbeit!!

flothi commented 7 years ago

Verrätst du auch was du umgebaut hast? ;-) Die Absenderkennung wäre interessant, guter Hinweis (und danke für die Blumen)

chbally commented 7 years ago

Umgebaut ist übertrieben... Ich habe die E-Mail Kontodaten getauscht, den Mailserver und den Port auf 25 wieder gewechselt. Ich weiß nicht wieviele Male ich diese Konfiguration erfolglos durchprobiert habe. :-( :-))

Schrolli91 commented 7 years ago

Also soweit kein Problem, sondern ein "Unfall" ? :-P

flothi commented 7 years ago

Seh ich ähnlich, der Hinweis mit dem Absender finde ich jedoch verfolgenswert

Schrolli91 commented 7 years ago

Dann sollte man dazu aber besser ein eigenes Issue aufmachen. @chbally it's your turn :-P