IoBroker.tankerkoenig | Spritpreis-Adapter

[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!

[ilovegym]

@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..

[ilovegym]

@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

[ilovegym]

@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

[ilovegym]

@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.

[root_]

Ist Tankerkönig nun ganz kaputt? Ich bekomme JSON returns error - Station ID or API-Key probably not correct und wenn ich auf der TK Homepage das log von meiner api laden will hab ich einen DB Error.

Was auch noch nicht so toll ist, wenn man den Adapter abschaltet bekommt man im Abfrageintervall Error Meldungen im Log.
instance system.adapter.tankerkoenig.0 terminated with code 3 (Adapter disabled or invalid config)

[Jan1]

Das liegt wohl nicht am Adapter sonder an der Seite. Versuch doch mal den API Key neu zu ordern, kommt gleich ne Fehlermeldung und die Seite ist auch recht träge.

[Homoran]

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

und die Seite ist auch recht träge.

weil die neue Version im Sekundentakt die API triggert ???

[ilovegym]

Bitte schaut, dass ihr die Version 2.0.4 benutzt, Versionen 2.0 bis 2.0.3 hatten noch Fehler.
Wahrscheinlich ist bei euch jetzt der Api-Key gesperrt worden, schaut mal in eure Email, dir ihr dort angegeben habt und lasst ihn wieder freischalten, nachdem! ihr das Update auf 2.0.4 gemacht habt.