(Gelöst) Curl und HMLAN Disconnected

[SamNitro]

Nor mal eine Anmerkung:
Da Du dieses Script schon mal als root (sudo) gestartet hast, könntest Du vorher prüfen, ob die Berechtigungen der Dateien stimmen?

Die Daten sind in Fhem angekommen falls du das meinst?!



Gesendet von iPhone mit Tapatalk

[SamNitro]

Dann mach zuerst:

Code Auswählen Erweitern

sudo chmod +x updatecheck.sh
Und dann NUR

Code Auswählen Erweitern

./updatecheck.shohne sudo.

Läuft jetzt auch,
und stehen 9 updates für fhem an.

[Wernieman]

Neee ..... die per wget runtergeldadenen Dateien gehören, wenn per sudo gestartet, root.

[SamNitro]

Dann weiß ich leider nicht was du meinst. Wie muss ich vorgehen?


Gesendet von iPhone mit Tapatalk

[Wernieman]

Sorry, habe mir erst jetzt gründlicher das Script durchgelesen ... es legt nichts im Dateisystem ab .. bist also "sauber"

[SamNitro]

Gut


Gesendet von iPhone mit Tapatalk

[amenomade]

Sooo... (Mit der Hilfe von betateilchen):

Code Auswählen Erweitern

#!/bin/bash
updatesfhem=$(curl -sS http://fhem.de/fhemupdate/controls_fhem.txt| diff - /opt/fhem/FHEM/controls_fhem.txt |grep ">"|grep "UPD"|wc -l)
updatesftui=$(wget -qO - https://raw.githubusercontent.com/knowthelist/fhem-tablet-ui/master/controls_fhemtabletui.txt| diff - /opt/fhem/FHEM/controls_fhemtabletui.txt |grep ">" |grep "UPD"|wc -l)
updatesall=$(($updatesfhem + $updatesftui))
perl fhem.pl localhost:7072 "setreading Update.Counter FHEM $updatesfhem; setreading Update.Counter FTUI $updatesftui; setreading Update.Counter ALL $updatesall"

textfhem=$(curl -sS http://fhem.de/fhemupdate/controls_fhem.txt| diff - /opt/fhem/FHEM/controls_fhem.txt  |grep "> UPD" )
echo -E setreading Update.Counter fhem_text $textfhem | sed 's/\n/ /g' | telnet localhost 7072

textftui=$(wget -qO - https://raw.githubusercontent.com/knowthelist/fhem-tablet-ui/master/controls_fhemtabletui.txt| diff - /opt/fhem/FHEM/controls_fhemtabletui.txt |grep "> UPD")
echo -E setreading Update.Counter ftui_text $textftui | sed 's/\n/ /g' | telnet localhost 7072

Sollte gehen.

Ich muss das aber noch optimieren, da wir damit 2x das gleiche aufrufen, einmal um die Anzahl und einmal um den Text zu kriegen. Muss aber jetzt weg.

[SamNitro]

Eilt ja nicht...


Aber jetzt gibt er mir eine Fehlermeldung raus:

Code Auswählen Erweitern

Trying ::1...
Trying 127.0.0.1...
Connected to localhost.
Escape character is '^]'.
Connection closed by foreign host.
Trying ::1...
Trying 127.0.0.1...
Connected to localhost.
Escape character is '^]'.
Connection closed by foreign host.


EDIT: Aber der text kommt trotzdem in Fhem an

[amenomade]

Ich vermute der fhem Server schützt sich gegen zu vielen Anfragen.
EDIT: wobei... das ist eher eine fhem Fehlermeldung

[amenomade]

So jetzt probier mal das (die Meldung kam von der Benutzung von telnet - war nicht schlimm, aber das hab ich geändert):

Code Auswählen Erweitern

#!/bin/bash

textfhem=$(curl -sS http://fhem.de/fhemupdate/controls_fhem.txt| diff - /opt/fhem/FHEM/controls_fhem.txt  |grep "> UPD" )
updatesfhem=`echo $textfhem |grep -o "UPD" |wc -l`
textfhem=${textfhem//[$'\n']}
perl /opt/fhem/fhem.pl localhost:7072 "setreading Update.Counter FHEM $updatesfhem; setreading Update.Counter fhem_text $textfhem"

textftui=$(wget -qO - https://raw.githubusercontent.com/knowthelist/fhem-tablet-ui/master/controls_fhemtabletui.txt| diff - /opt/fhem/FHEM/controls_fhemtabletui.txt |grep ">" |grep "UPD")
updatesftui=`echo $textftui |grep -o "UPD" |wc -l`
textftui=${textftui//[$'\n']}
perl /opt/fhem/fhem.pl localhost:7072 "setreading Update.Counter FTUI $updatesftui; setreading Update.Counter ftui_text $textftui"

updatesall=$(($updatesfhem + $updatesftui))
perl /opt/fhem/fhem.pl localhost:7072 "setreading Update.Counter ALL $updatesall"


Testen wie immer mit ./updatecheck.sh
Wenn es geht, installieren wir es in crontab, damit es jeden Tag um 9:00 Uhr läuft (mehr braucht man nw. nicht, da die eingecheckte Updates nur einmal am Nacht zur Verfügung gestellt werden - wir wollen nicht die Server von fhem.de unnötig belasten).

Code Auswählen Erweitern

crontab < <(crontab -l ; echo "0 9 * *  * pi /opt/fhem/updatecheck.sh")

[SamNitro]

Hier das resultat:

Code Auswählen Erweitern

pi@raspberrypi:/opt/fhem $ ./updatecheck.sh
Usage: setreading <name> <reading> <value>
where <name> is a single device name, a list separated by komma (,) or a regexp. See the devspec section in the commandref.html for details.


Code Auswählen Erweitern

pi@raspberrypi:/opt/fhem $ crontab < <(crontab -l ; echo "0 9 * *  * pi /opt/fhem/updatecheck.sh")
no crontab for pi

Edit: ist aber trotzdem eingetragen

Code Auswählen Erweitern

pi@raspberrypi:/opt/fhem $ crontab -l
0 9 * *  * pi /opt/fhem/updatecheck.sh


die readings kommen aber an..

Sorry für die Umstände

[SamNitro]

kann es sein das er sich an den ">" aufhängt und die nicht übertragen kann?

Code Auswählen Erweitern

> UPD 2017-08-22_07:45:03 207569 ./CHANGED

[SamNitro]

Habe es am Laufen

Code Auswählen Erweitern

setreading Update.Counter fhem_text $textfhem
setreading Update.Counter ftui_text  $textftui

habe ich ersetzt durch

Code Auswählen Erweitern

setreading Update.Counter fhem_text FHEM$textfhem
setreading Update.Counter ftui_text  FTUI$textftui

da ist was am anfang was den stört (>), jetzt sieht das so auf dem Tablet aus:



Alles PERFEKT

[Wernieman]

@amenomade

Ich hätte zu Deinem Script 2 Anmerkungen:

1.
Ich würde Dir empfehlen, anstatt telnet (oder fhem) nc zu verwenden.

nc kannst Du analog zu telnet, z.B.

Code Auswählen Erweitern

echo -E setreading Update.Counter fhem_text $textfhem | sed 's/\n/ /g' | nc localhost 7072

Allerdings wird es schwirig, wenn bei telnet ein Passwort vergeben wurde. Aber dieses kann man auch übergeben.

2. Anstatt für jede Anfrage eine eigene Session zu fhem aufzubauen, würde ich es bündeln.
Ala:

Code Auswählen Erweitern

echo -en "fhem-Befel 1\nfhem-Befehl2\nfhem-Befehl3\nquit\n | nc localhost 7072
Übrigens kann man dann auch das Passwort mit übergeben, indem es an erster Stelle gestellt wird, sofern vergeben ...

Ergebnis:
Also in Summe würde ich anstatt aller perl fhem-Aufrufe

Code Auswählen Erweitern

echo -en "setreading Update.Counter FHEM $updatesfhem\nsetreading Update.Counter fhem_text $textfhem\nsetreading Update.Counter FTUI $updatesftui\nsetreading Update.Counter ftui_text $textftui\nsetreading Update.Counter ALL $updatesall\nquit" | nc -w5 localhost 7072 >/dev/null

error=$?
if [ "$error" -ne 0 ]
then
  echo "`date` : Fehler $error beim Aufruf \"$0 $@\""
  exit 1
fi


Das "-w 5" damit er max 5 sec wartet ...

Hat den Vorteil, das nur 1. Verbindung zu FHEM aufgemacht wird ... und habe Dir gleich eine Fehlerabfangroutine eingebaut ...

(P.S. Man kann es auch im perl-fhem-Aufruf zusammenfassen, hattest Du mit ";" schon gemacht)

Edit:
Die Änderung von SamNitro sind nicht drin.


Edit2:
Zur Besseren Lesbarkeit kann man den Befehl auch einzeln zusammenbauen

Code Auswählen Erweitern


FemBefehl=""
FemBefehl="${FemBefehl}fhem-Befel1\n"
FemBefehl="${FemBefehl}fhem-Befel2\n"
FemBefehl="${FemBefehl}fhem-Befel3\n"
FemBefehl="${FemBefehl}quit\n"
echo -en "${FemBefehl}" | nc localhost 7072

[SamNitro]

WOW das überfordert mich wieder

und ja Passwort ist vergeben..