UPDATE 31.10.: Amazon Alexa - ioBroker Skill läuft aus ?

[Thomas Braun]

@martin-0

Zeig doch mal konkret deine Datei.

[G.Hawk]

@thomas-braun

[Thomas Braun]

@martin-0

Bitte keine Screenshots und den Raspberry NICHT MIT DESKTOP BETREIBEN!

[G.Hawk]

@thomas-braun

OK wie mache ich das denn richtig?

[Thomas Braun]

@martin-0

Per ssh in einem Terminal auf den Pi gehen. puTTY oder die Windows Powershell nimmt man dafür z. B.
Den Pi betreibt man im sog. RunLevel 3.

[viper4iob]

@martin-0 said in [Linux Shell-Skript] WLAN-Wetterstation:

@thomas-braun

Das sieht mir nach einem Formatierungsproblem aus. Da sind mir zu viele Leerzeilen drin.
Direkt nach den Überschriften sollten die Argumente kommen, da ist aber überall eine Leerzeile, evtl. hat der systemd ein Problem damit.

[Thomas Braun]

@viper4iob

Oder die Datei ist mit einem Windows-Editor vermurkst worden und hat keine Linux-Zeilenumbrüche.

[G.Hawk]

@viper4iob
Ich habe jetzt alle Leerzeichen gelöscht.
Keine Veränderung

[G.Hawk]

@thomas-braun
Den habe ich garnicht benutzt.

[G.Hawk]

Deutet der Fehler nicht eher daraufhin, dass die Datei garnicht gefunden wird bzw nicht ausführbar ist?

[Thomas Braun]

@martin-0

Dann schau nach:

ls -la /home/iobroker/wetterstation.sh

[G.Hawk]

pi@raspberrypi:~ $ ls -la /home/iobroker/wetterstation.sh
-rwxr-xr-x 1 root root 15472 Nov  3 09:40 /home/iobroker/wetterstation.sh

---

Mod-Edit: Code in </> Code-Tag gepackt!

[G.Hawk]

@martin-0
Es lag an der Formatierung.

Ich habe Leerzeilen eingefügt zwischen den Befehlen. Jetzt geht es.

[viper4iob]

@martin-0
Das klingt dann danach, dass die Zeilenumbrüche nicht gepasst haben.
Linux nutzt LF und Windows CRLF und wenn man das irgendwie von Windows rüber holt, hat man evtl. die falschen.
Da gibt es dann z.B. das Kommando dos2unix, um das umzuwandeln.
Und wenn man die Zeilenumbrüche he dem Editor unter Linux wieder einfügt, nimmt er natürlich LF
Du hast also vorhin beim Löschen der Leerzeilen wahrscheinlich die falschen gelöscht und dann durchs einfügen, die richtigen hinzugefügt.

[Thomas Braun]

@martin-0

Die Datei in /home/iobroker sollte aber NIE dem root gehören.

[G.Hawk]

@thomas-braun
Ich habe alles mögliche umgestellt.
Warum denn nicht?

[Thomas Braun]

@martin-0
Weil das das /home des iobrokers ist. Und nicht vom root. Der wohnt unter /root.

[viper4iob]

@martin-0
Kannst du mal das machen :
cat -e /etc/systemd/system/wetterstation.service

Der sollte den Inhalt der Datei mit Zeilenumbrüchen darstellen. Ich hoffe auch CRLF, falls vorhanden.
Was kommt da raus?
Es geht zwar jetzt, aber diese Leerzeilen gehören da eigentlich nicht hin. Evtl. hast du jetzt beide Typen drin.

EDIT
Er müsste $ für LF anzeigen, was richtig wäre.
Und ^M$ für CRLF. Die sollten da nicht auftauchen.

[Thomas Braun]

@martin-0

sudo chown iobroker:iobroker /home/iobroker/wetterstation.sh

[viper4iob]

Zum Thema Dateirechten und Besitzerschaft vom Skript:
Im Prinzip gebe ich @Martin-0 recht. Wenn man das Skript im Home vom iobroker betreibt, sollte es auch iobroker gehören.
Aus Sicherheitssicht spielt es aber keine große Rolle, denn sobald das Skript als systemd Service gestartet wird, läuft es unter root. Kann man beispielsweise mit ps -ef | grep wetterstation kontrollieren.
Und das Risiko ist denke ich deutlich höher, falls eben jemand das Github Repository manipuliert und man sich diese Änderungen dann installiert.

Wichtig wäre also meiner Meinung nach den Service unter einem anderen User als root ausführen zu lassen. Das Skript braucht soweit ich das aktuell feststellen konnte, für das normale Ausführen keine root-Rechte, es führt netcat aus und solange der Port über 1024 ist, braucht das keine root Rechte. Und dann kommuniziert es mit iobroker und die InfluxDB über deren Schnittstellen, was ebenfalls keine root Rechte benötigt.
Um den Service als anderer User zu starten muss man im Service File im Abschnitt [Service] die Argumente User und Group hinzufügen und da dann beispielsweise den iobroker angeben.
Ich hab es aktuell unter meinem eigenen User laufen.
Streng genommen sind beide User nicht optimal und man müsste einen komplett neuen User nur für das Skript anlegen, der sich auch nicht interaktiv anmelden kann und darüber dann den Dienst laufen lassen, aber das war mir dann auch zu viel des Guten.
Unter meinem User funktioniert es auf alle Fälle.
Über den iobroker User habe ich es noch nicht getestet. Der iobroker user wäre vielleicht aktuell sogar die bessere Wahl, weil der sich wahrscheinlich nicht interaktiv anmelden kann, also nur ein System User ist. Nur weiß man nie, ob sich da mit irgendeinem Update mal wieder was ändern könnte. Bis vor einiger Zeit lief der iobroker auch nicht unter eigenem User, das wurde dann irgendwann mal angepasst.

@SBorg
Das wäre evtl. nochmal ein Punkt, worüber man für den Install Guide und der Service File Definition nachdenken könnte. Vielleicht auch einfach nur als Hinweis und man kann dann selbst entscheiden.