IoBroker.tankerkoenig | Spritpreis-Adapter

[A Former User]

@pix Der Abfrageintervall der Version 2.0.1 bringt gerade die Server von Tankerkönig ins Schwitzen, bitte alle, die Version 2.0.1 installiert haben, überprüfen wie oft eure Abfrage wirklich stattfindet ( https://creativecommons.tankerkoenig.de/ hier könnt ihr euch euer Logfile ansehen ) und den Adapter wieder downgraden, bis der Bug behoben ist.

Issue auf Github hab ich schon aufgemacht. ( https://github.com/Pix---/ioBroker.tankerkoenig/issues/28 )

Ilovegym66 created this issue in Pix---/ioBroker.tankerkoenig

closed 2.0.1 - too much requests #28

[pix]

Danke für den Hinweis. Stoppt Eure Instanz oder downgraded bitte.

Pix

[arteck]

setzt doch mal Abfrageintervall min alle 10 min.. intern.. egal was der User eingibt

[Homoran]

@arteck
Das nützt auch nichts, wenn 20000 ioBroker User zeitgleich auf den Server zugreifen gibt's auch Stress

[arteck]

@Homoran
du greifst nicht innerhalb von einer milisekunde mit 20000 Users auf die API drauf...

es geht darum dass falls mehrere tausend User das Abfrageinterwall auf 1 min. setzten das man das unterbindet..

zur Zeit laufen die Abfragen sekündlich

[Segway]

Man sollte sich schon selber fragen wie sinnvoll es ist etwas abzufragen.
Die Spritpreise ändern sich doch nicht so schnell, so dass eine Abfrage zwischen 10-20 Mins vollkommen ausreichend ist. Hinzu kommt, das es in der Vergangenheit sehrwohl zu Problemen kam, als wahrscheinlich ein Großteil der User mit der Standardeinstellung von 5 Mins gearbeitet hatte. Da kamen dann keine Werte mehr.
Wenn man dann irgendwas krummes einstellt (alle 9 Minuten zur Sekunde 43) und jeder was individuelles für sich einstellt dann klappt es auch wieder !
Ansonsten mal die AGB's lesen im bereich "Kommerzieller Service", da man bei Nutzung einer API auch als User Verpflichtungen eingeht !

[Homoran]

@Segway sagte in IoBroker.tankerkoenig | Spritpreis-Adapter:

Man sollte sich schon selber fragen wie sinnvoll es ist etwas abzufragen.

Ich denke das klappt schon ganz gut.
In der neuen Version ist es ein Bug!

[A Former User]

@Homoran @Segway
richtig, es ist ein Bug, ich hatte 15min eingestellt und der Adapter hat 150 Abfragen pro Sekunde gemacht!
Das ist doch schon ein kleiner Unterschied..?

[Segway]

@Homoran sagte in IoBroker.tankerkoenig | Spritpreis-Adapter:

In der neuen Version ist es ein Bug!

Das ist richtig - trotzdem wollte ich zu diesem Thema einmal sensibilisieren (generell( sich einmal Gedanken zu amchen, was Sinn macht und was nicht.

@ilovegym sagte in IoBroker.tankerkoenig | Spritpreis-Adapter:

Das ist doch schon ein kleiner Unterschied..?

Korrekt !

[pix]

Ich gucke mir ab jetzt nochmal den Code an, konnte aber bei meinen Tests den Effekt nicht feststellen. Das Log hat nur alle 7 Minuten die Abfrage protokolliert.

Es braucht keine "krumme" Zeit, da jede Installation zu unterschiedlichen Zeiten gestartet wurde. Sie ist nicht an zB 0:00:00 Uhr gebunden.

In den Adaptereinstellungen ist es nicht möglich, ein Intervall unter 5 Minuten abzuspeichern (siehe rote Linie)

@Segway Die Nutzung des Adapters ist privat und nicht kommerziell. Durch den eigenen API-Key ist jeder selbst verantwortlich. Der Adapter ist nur ein Hilfsmittel. Es steht jedem frei, zB das Javascript (hier aus dem Forum) zu verwenden oder einen eigenen Adapter zu schreiben.

Ich seh mir das also nochmal an.

Pix

[SBorg]

Sekundentakt kann ich nicht bestätigen, allerdings hatte ich 16 Minuten eingestellt, fragt aber alle ~5 Minuten ab.

xx.xx.xx.xx - - [21/Feb/2019:09:59:50 +0100] "GET /json/prices.php?ids=... ... HTTP/1.1" 200 496
xx.xx.xx.xx - - [21/Feb/2019:10:04:50 +0100] "GET /json/prices.php?ids=... ... HTTP/1.1" 200 496
xx.xx.xx.xx - - [21/Feb/2019:10:10:09 +0100] "GET /json/prices.php?ids=... ... HTTP/1.1" 200 496
xx.xx.xx.xx - - [21/Feb/2019:10:15:09 +0100] "GET /json/prices.php?ids=... ... HTTP/1.1" 200 496
xx.xx.xx.xx - - [21/Feb/2019:10:22:13 +0100]...

[arteck]

schau mal ob du die Zeile im Log findest die mit

Sync set to

anfängt
müsste unter info stehen..und direkt nach adapter start..

[A Former User]

@arteck hier das Problem:
2019-02-21 10:40:21.582 - debug: tankerkoenig.0 objectDB connected
2019-02-21 10:40:21.595 - debug: tankerkoenig.0 statesDB connected
2019-02-21 10:40:21.966 - info: tankerkoenig.0 starting. Version 2.0.2 in /opt/iobroker/node_modules/iobroker.tankerkoenig, node: v8.15.0
2019-02-21 10:40:21.979 - info: tankerkoenig.0 Sync set to undefinedmin or NaN ms
2019-02-21 10:40:21.980 - debug: tankerkoenig.0 Option <reset values> is false
2019-02-21 10:40:21.980 - debug: tankerkoenig.0 API Key Länge: 36 Zeichen

Obwohl in im Admin Adapter 17 Minuten eingestellt habe....

[arteck]

@ilovegym sagte in IoBroker.tankerkoenig | Spritpreis-Adapter:

info: tankerkoenig.0 Sync set to undefinedmin or NaN ms

Pix der übernimmt nicht die

adapter.config.synctime

in zeile 513

[pix]

Hallo arteck, danke für den Hinweis. Habe das schon gesehen, in der Admin-Datein index_m.html wurde lokal ein Unterstrich verwendet und in der main.js ohne Unterstrich abgefragt. Das konnte nicht gehen. So etwas übersieht man schnell, auch mit Brille

ilovegym hat auf github auch mit Testergebnissen geholfen. Jetzt geht es. Müssen mal abwarten, ob die Jungs von Tankerkönig nochmal meckern.

Ich empfehle, wie schon auf Github erklärt, unbedingt die Instanz und dann die Objekte zu löschen, wenn man eine 2er-Version installiert hatte. Bei der Umstellung von schedule auf daemon blieben trotzdem "schedule-Reste" im System, die sich nur so entfernen ließen.
Durch die Admin3-Funktion "Adapter Einstellungen speichern und laden" ist das saubere Installieren kein Problem mehr.

2.0.3 auf Github bereit

Gruß
Pix

[pix]

Achso, eins noch: das Intervall lässt sich zZt im Log noch nicht so richtig nachvollziehen. Ich korrigiere das, wenn das Hauptproblem Intervall gelöst wurde. Aber der Timestamp der Objekte eignet sich.

Pix

[A Former User]

@pix also 9 minuten geht, steht auch so im log:

2019-02-21 11:15:49.379  - info: tankerkoenig.0 starting. Version 2.0.3 in /opt/iobroker/node_modules/iobroker.tankerkoenig, node: v8.15.0
2019-02-21 11:15:49.392  - info: tankerkoenig.0 Sync set to 9min or 540000 ms
2019-02-21 11:15:49.456  - info: tankerkoenig.0 Daten werden eingelesen
2019-02-21 11:15:49.462  - info: tankerkoenig.0 Adapter gestoppt - es werden keine Daten mehr eingelesen
2019-02-21 11:15:49.379  - info: tankerkoenig.0 starting. Version 2.0.3 in /opt/iobroker/node_modules
2019-02-21 11:24:49.486  - info: tankerkoenig.0 Daten werden eingelesen
2019-02-21 11:24:49.487  - info: tankerkoenig.0 Adapter gestoppt - es werden keine Daten mehr eingelesen

[pix]

Ja, genau. Die Log Meldung über die Intervallgröße kommt nach dem Einlesen der Settings (es wird eine andere Meldung, wenn das Intervall unter 5 Minuten sein sollte). Aber die log-Zeiten sagen ja auch 9min aus.

Die Meldung "Adapter gestoppt - es werden keine Daten mehr eingelesen" wird in der nächsten Version rausgenommen. Sie ist falsch und steht auch an falscher Stelle im Code.

Vielen Dank fürs Test und fürs Feedback. So konnten wir das DIn innerhalb von ein paar Stunden wieder flot(t) machen

Gruß
Pix

[A Former User]

@pix kein Thema, gern geschehen! Glaub die 2.0.4 braucht n push auf npm.. kann das sein??

[pix]

Yep, publish 2.0.4 auf npm kommt. Travis und Appveyor testen aber gerade noch.