[Frage] Einfaches Script verursacht JS Absturz - Warum

mickyboys

OK, werde es mal testen

baennjer

Habe jetzt in javascript 3.3.4 ein Fix dafür. `

Hallo Bluefox, ich habe JavaScript 3.3.8 installiert aber der Adapter schmiert trotzdem ab wenn was mit der URL nicht stimmt. Es kommt manchmal vor wenn ich die Wetterdaten an Wunderground.com schicke dass die nicht erreichbar sind und schon ist der Adapter im Restart.

Muss man irgendwas beachten beim REQUEST?

Blackeye

Hallo zusammen,

das Problem gibt es mit Version 3.4.0 noch immer:

avascript.0 ReferenceError: e is not defined at Request. (script.js.common.Aktionen.Aktion_aus_wenn_nicht_gebraucht:4:116) at emitOne (events.js:96:13) at Request.emit (events.js:188:7) at Request.onRequestError (/opt/iobroker/node_modules/iobroker.javascript/node_modules/request/request.js:878:8) at emitOne (events.js:96:13) at ClientRequest.emit (events.js:188:7) at Socket.socketOnEnd (_http_client.js:345:9) at emitNone (events.js:91:20) at Socket.emit (events.js:185:7) at endReadableNT (_stream_readable.js:974:12)

Und bei mir kommt es auch nur zu dem Fehler falls die Adresse die angesteuert werden soll nicht mehr erreichbar ist.

Gruß

Lars

apollon77

Wie sieht denn Der js Code aus?

Dutchman

~~@Blackeye:~~

Und bei mir kommt es auch nur zu dem Fehler falls die Adresse die angesteuert werden soll nicht mehr erreichbar ist. `

Das war doch eigentlich teil eines fixes in version 3.3.4?

3.3.4 (2017-04-01)

(bluefox) Catch error by request if host unavailable
(bluefox) add "request" to script namespace

Hat sich der fehler ab ein bestimmten version eingeschlichen ?

Blackeye

Hi,

kann ich nicht sagen, da ich neu in dem Thema bin. Aber es ist genau der beschriebene Fehler:

Der deaktivierte Teil verursacht den Fehler…..

apollon77

Aaahhh Blockly … das fehlt im Thread Namen!!

Dutchman

~~@Blackeye:~~

Hi,

kann ich nicht sagen, da ich neu in dem Thema bin. Aber es ist genau der beschriebene Fehler:

Unbenannt.PNG

Der deaktivierte Teil verursacht den Fehler….. `

Koenntest du mal deinen javascript adapter downgraden nach 3.4.4 oder 3.3.6 und chaun ob das problem dan auch noch da ist ?

Experten modus aktivieren :

Version downgraden :

Blackeye

Hi,

bei beiden Versionen folgende Fehler:

<code>host.ioBroker-RasPi	2017-10-29 10:31:04.189	error	instance system.adapter.javascript.0 terminated with code 0 (OK)
javascript.0	2017-10-29 10:31:04.096	error	ReferenceError: e is not defined at Request. (script.js.common.Aktionen.Aktion_aus_wenn_nicht_gebraucht:4:116) at emitOne (events.js:96:13) at Request.emit (events.js:188:7)
javascript.0	2017-10-29 10:31:04.096	error	uncaught exception: e is not defined
javascript.0	2017-10-29 10:31:04.096	error	ReferenceError: e is not defined at Request. (script.js.common.Aktionen.Aktion_aus_wenn_nicht_gebraucht:4:116) at emitOne (events.js:96:13) at Request.emit (events.js:188:7)
javascript.0	2017-10-29 10:31:04.096	error	uncaught exception: e is not defined
[code][/code]</code>

~~@Dutchman:~~

~~@Blackeye:~~

Koenntest du mal deinen javascript adapter downgraden nach 3.4.4 oder 3.3.6 und chaun ob das problem dan auch noch da ist ?

Gruß

Lars

apollon77

Bitte

1.) Das Blockly auf machen, irgend eine kleine Änderung machen das der Javascript-code neu generiert wird

2.) ann schauen ob es wieder passiert

3.) Wenn ja, generiertes JavaScript (Zeige Script) posten

Blackeye

Hi,

habe alles gemacht nd bekomme die gleichen Fehler, daher hier der Code…

schedule("* * * * *", function () {
  if (getState("harmony.0.Stube.activities.currentActivity").val != 'Fernseher') {
    try {
      require("request")('http://192.168.188.30/web/powerstate?newstate=1').on("error", function () {console.error(e);});
    } catch (e) { console.error(e); }
    console.log("request: " + 'http://192.168.188.30/web/powerstate?newstate=1');
  }
  if (getState("harmony.0.Stube.activities.currentActivity").val == 'PowerOff') {
    setState("harmony.0.Stube.Sony-Fernseher.PowerOff"/*Sony-Fernseher:PowerOff*/, 1);
  }
});

~~@apollon77:~~

Bitte

1.) Das Blockly auf machen, irgend eine kleine Änderung machen das der Javascript-code neu generiert wird

2.) ann schauen ob es wieder passiert

3.) Wenn ja, generiertes JavaScript (Zeige Script) posten `

pix

Hallo,

kenn mich nicht Blockly gar nicht aus, aber ~~@Blackeye:~~

schedule("* * * * *", function () {
  if (getState("harmony.0.Stube.activities.currentActivity").val != 'Fernseher') {
    try {
      require("request")('http://192.168.188.30/web/powerstate?newstate=1').on("error", function () {console.error(e);});
    } catch (e) { console.error(e); }
    console.log("request: " + 'http://192.168.188.30/web/powerstate?newstate=1');
  }
  if (getState("harmony.0.Stube.activities.currentActivity").val == 'PowerOff') {
    setState("harmony.0.Stube.Sony-Fernseher.PowerOff"/*Sony-Fernseher:PowerOff*/, 1);
  }
});
```` `

Zeile 4 sieht mir nicht valide aus.

Blackeye

~~@pix:~~

Hallo,

kenn mich nicht Blockly gar nicht aus, aber ~~@Blackeye:~~

schedule("* * * * *", function () {
  if (getState("harmony.0.Stube.activities.currentActivity").val != 'Fernseher') {
    try {
      require("request")('http://192.168.188.30/web/powerstate?newstate=1').on("error", function () {console.error(e);});
    } catch (e) { console.error(e); }
    console.log("request: " + 'http://192.168.188.30/web/powerstate?newstate=1');
  }
  if (getState("harmony.0.Stube.activities.currentActivity").val == 'PowerOff') {
    setState("harmony.0.Stube.Sony-Fernseher.PowerOff"/*Sony-Fernseher:PowerOff*/, 1);
  }
});
```` `

Zeile 4 sieht mir nicht valide aus. `

Aber da ich nur mit Blockly gearbeitet habe, spricht ja etwas für einen Fehler.

Danke

apollon77

Also in der aktuellen Version des JavaScript Adapters sollte der Code korrekt erzeugt werden:

            return 'try {\n  require("request")(' + URL + ').on("error", function (e) {console.error(e);});\n} catch (e) { console.error(e); }\n' +
                logText;

Exportiere mal das Blockly, lösche dasd Srkitp und erstelle neu und importiere. Und sicherstellen das Javascript Adapter aktuell ist. AM besten von Github installieren

Blackeye

Hi,

alles gemacht aber das Ergebnis bleibt. Ist es möglich das irgendwie anders zu lösen?

~~@apollon77:~~

Also in der aktuellen Version des JavaScript Adapters sollte der Code korrekt erzeugt werden:
            return 'try {\n  require("request")(' + URL + ').on("error", function (e) {console.error(e);});\n} catch (e) { console.error(e); }\n' +
                logText;
Exportiere mal das Blockly, lösche dasd Srkitp und erstelle neu und importiere. Und sicherstellen das Javascript Adapter aktuell ist. AM besten von Github installieren `

blauholsten

Hi,

ich verfolge die letzten Tage diesen Thread, nun will ich doch mal versuchen etwas beizutragen bzw. eine Idee einzubringen. Bitte haut nicht auf mich ein, falls der folgende Text kompletter Quatsch ist, ich habe nur einige Grundkenntnisse von JS. (Von Blocky gar keine)

Für mich ist der Programmcode falsch, denn TRY / CATCH kann keine asyncronen Aufrufe verarbeiten. Im vorliegenden Fall ist der Request aber Asyncron, und der auftretende Fehler tritt auf wenn try schon längst abgearbeitet ist.

Wenn man hier den TRY / CATCH Block weglässt, wird die Instanz nicht mehr abstürzen, denn die ERROR Behandlung wird über````
.on("error“,……


DAS IST NUR EINE VERMUTUNG; ALSO BITTE NICHT LACHEN…….

blauholsten

Keine Antwort von den Profis…. :oops:

AlCalzone

~~@blauholsten:~~

Wenn man hier den TRY / CATCH Block weglässt, wird die Instanz nicht mehr abstürzen, denn die ERROR Behandlung wird über.on("error“,…… abgefangen. `

Ne das ist Quatsch. on("error") ist keine Fehlerbehandlung an sich, sondern gibt Zugriff auf Fehler, die im Request-Modul bereits abgefangen wurden. Try/Catch verhindert Adapterabstürze durch Fehler, die davon nicht abgefangen wurden. Mit dem asynchronen hast du aber Recht. Weglassen von try/catch wird nicht dafür sorgen, dass Abstürze plötzlich nicht mehr zum Absturz führen (außer in catch sind neue Fehler eingebaut)

blauholsten

~~@AlCalzone:~~

~~@blauholsten:~~

Wenn man hier den TRY / CATCH Block weglässt, wird die Instanz nicht mehr abstürzen, denn die ERROR Behandlung wird über.on("error“,…… abgefangen. `

Ne das ist Quatsch. on("error") ist keine Fehlerbehandlung an sich, sondern gibt Zugriff auf Fehler, die im Request-Modul bereits abgefangen wurden. Try/Catch verhindert Adapterabstürze durch Fehler, die davon nicht abgefangen wurden. Mit dem asynchronen hast du aber Recht. Weglassen von try/catch wird nicht dafür sorgen, dass Abstürze plötzlich nicht mehr zum Absturz führen (außer in catch sind neue Fehler eingebaut) `
Hi,

sorry ich habe mich verkehrt ausgedrückt, ich meinte damit Fehler im request wie UNREACH usw….

habe mir auf der ersten Seite die Fehler nachmal angeschaut, es scheint ja so als ob im callback "e" nicht übergeben wurde....

Hier hilft try/catch nicht, da der callback asynchron aufgerufen wird.

 .on("error", function () {console.error(e);});

aber die Entwickler haben das anscheinend behoben, sodass das eigentlich behoben ist :oops:

Blackeye

Hi,

ich habe für mich eine Lösung gefunden.

Ich habe den Ping Adapter installiert und frage die Erreichbarkeit des Gerätes ab.

Gruß

Lars

Community Forum

NEWS

[Frage] Einfaches Script verursacht JS Absturz - Warum

Support us

613

32.6k

82.0k

1.3m