[gelöst] auf globale Objekt kann nicht zugegriffen werden

[Ben1983]

@haus-automatisierung OK. Danke

Beim Verwendet von messageToAsynch tritt dieser Fehler auf:

messageToAsync("Testmessage",{name:"Testname"});

	javascript.0 (20591) script.js.x_testskripte._a_Mesagetest: TypeError: Cannot read properties of undefined (reading 'timeout')

[paul53]

@ben1983 sagte: "Besser" das Objekt nur dazu definieren, wo es gebraucht wird?

Ja. Die Gruppe "global" verwendet man für häufig verwendete eigene Funktionen.
Mit globalen Objektdefinitionen müllt man sich den RAM der Javascript-Instanz zu, da sie in jedem nicht globalen Skript kompiliert werden.

[haus-automatisierung]

@ben1983 sagte in auf globale Objekt kann nicht zugegriffen werden:

Beim Verwendet von messageToAsynch tritt dieser Fehler auf:

Ist wohl nen Bug wenn die Options fehlen. So gehts:

const res = await messageToAsync('Testmessage', { flag: true }, { timeout: 5000 });
console.log(JSON.stringify(res));

Machst einen GitHub Issue?

[ticaki]

@ben1983 sagte in auf globale Objekt kann nicht zugegriffen werden:

messageToAsync

ups falsch gelesen

EDIT aber trotzdem relevant und die Beschreibung/Definition sollte geändert werden.

function messageToAsync(target: iobJS.MessageTarget | string, data: any, options?: any): Promise<iobJS.MessageCallback | iobJS.MessageCallbackInfo>;

Der {timeout:30000} (zeit is nur ein Beispiel) sollte im Javascript-Adapter zumindest ein default sein oder die Angabe verpflichtend wie jetzt. Ohne timeout kann das an der Stelle hängen bleiben, wie ich schmerzlich feststellen mußt mit einem gleichzeitigen Defekt des Übertemperaturwächters am Boiler.

[haus-automatisierung]

@ticaki sagte in auf globale Objekt kann nicht zugegriffen werden:

Ohne timeout kann das an der Stelle hängen bleiben, wie ich schmerzlich feststellen mußt

Habe ich gerade das erste mal von gehört. Gibts dazu schon einen GitHub Issue? Sollte zumindest in der nächsten Version gefixt sein:

https://github.com/ioBroker/ioBroker.javascript/commit/41e59dfd72f99027a9a303beee8d4d5597826445

0 klein0r committed to ioBroker/ioBroker.javascript

messageToAsync was not working without options

[ticaki]

@haus-automatisierung
Ach mist, nicht das es nicht das gleiche ist. Bei mir ist sendToAsync hängen geblieben. Sorry hab die beiden durcheinandergewürfelt.

[haus-automatisierung]

@ticaki sendToAsync funktioniert bei mir wunderbar

[ticaki]

@haus-automatisierung sagte in auf globale Objekt kann nicht zugegriffen werden:

@ticaki sendToAsync funktioniert bei mir wunderbar

Stoppe den Zieladapter und nutzte dann "sendToAsync" ohne timeout.

[haus-automatisierung]

@ticaki Bei sendToAsync gibts keinen Timeout - eventuell ist das auch dein Problem?

[ticaki]

@haus-automatisierung sagte in auf globale Objekt kann nicht zugegriffen werden:

@ticaki Bei sendToAsync gibts keinen Timeout - eventuell ist das auch dein Problem?

Das ist ja noch schlimmer, ist mir garnicht aufgefallen, dass das nicht durchgereicht wird. Ohne Temperaturwächter setzte ich kein sendToAsync mehr für Warmwassererhitzung ein.

Beide SendtoAsync bleiben hängen, wenn telegram ausgeschaltet ist.

test1()
test2()
async function test1() {
    await sendToAsync("telegram", "send", {user: 'Tim', text: 'Test1'});
    log('test1');
}
async function test2() {
    await sendToAsync("telegram", "send", {user: 'Tim', text: 'Test2'}, { timeout: 1000 });
    log('test2');
}

Das hier in meinem Adapter jedoch nicht:

await this.adapter.sendToAsync(this.options.adapter, 'send', opt, { timeout: 2000 })

https://github.com/ioBroker/ioBroker.javascript/issues/1436

ticaki created this issue in ioBroker/ioBroker.javascript

closed Missing timeout for sendToAsync() #1436

[haus-automatisierung]

Die Funktion hat nur 3 Parameter, alle weiteren werden ignoriert:

https://github.com/ioBroker/ioBroker.javascript/blob/41e59dfd72f99027a9a303beee8d4d5597826445/lib/sandbox.js#L2702

@ticaki sagte in auf globale Objekt kann nicht zugegriffen werden:

Das hier in meinem Adapter jedoch nicht:

Im der Adapter-Klasse gibt es noch einen weiteren, richtig:

https://github.com/ioBroker/ioBroker.js-controller/blob/0b01593055f5ebb61c1a9939a2a962b7b53f7344/packages/adapter/src/lib/adapter/adapter.ts#L7106

[ticaki]

@haus-automatisierung
Um wieder etwas on topic zu werden.

Der timeout sollte(geht halt aktuell nur bei messageToAsync) bei beiden ToAsync gesetzt werden, wenn diese mit await genutzt werden, den wenn der Callback wegen gestopptem Skript/Adapter nicht beantwortet wird, bleibt das Skript an der Stelle stehen.

[haus-automatisierung]

@ticaki Logisch, ist bis dato nur scheinbar niemandem aufgefallen (bzw. wurde nicht gemeldet)

[Ben1983]

@haus-automatisierung irgendwie will es heute nicht...

wieso bekomme ich jetzt keinen Log?

log(JSON.stringify( await messageToAsync("Testmessage",{name:"Testname"},{timeout:5000})));

onMessage("Testmessage",(messagedata,callback) =>{
    const myData = {input: messagedata,
                    output:{name:"Test"}};
    callback(myData)
});

[ticaki]

@haus-automatisierung
Wäre mir wohl so auch nicht klar gewesen aber erste habe ich wegen hängendem sendToAsync im Adapter erfahren dass es da keinen default timeout gibt. Und 2 Tage später fliegen die Leitungen von meinem Boiler und 4 Stunden vorher haben die Logeinträge meines Warmwasserskripts geendet. Da war der Zusammenhang nicht schwer herzustellen. Mist nur das die Überhitzungssicherheitsschaltung zu alt/ kaputt war. Naja, war zum Glück nur ne Riesensauerei.

[ticaki]

@ben1983

Das await müsste bei dir eigentlich als Fehler markiert sein.

test()

async function test() {
    log(JSON.stringify( await messageToAsync("Testmessage",{name:"Testname"},{timeout:10000})));
}
 
onMessage("Testmessage",(messagedata,callback) =>{
    const myData = {input: messagedata,
                    output:{name:"Test"}};
    callback(myData)
});

Ach und mit abgestürzter Javascriptinstanze gibts keine logs

[Ben1983]

@haus-automatisierung erledigt:
https://github.com/ioBroker/ioBroker.javascript/issues/1437

BenAhrdt created this issue in ioBroker/ioBroker.javascript

closed messageToAsync funktionier nicht ohne options (ohne timeout) #1437

[Ben1983]

@ticaki ok und woran liegt es?
Es wird der Toplevel angeckert... die war aber schon länger so undmachte eigentlich keine Probleme.
Woran liegt jetzt der Fehler?
Habe ich was übersehen?

@haus-automatisierung kannst Du mir sagen, warum es nicht geht?

bei deinem Beispeil:

const res = await messageToAsync('Testmessage', { flag: true }, { timeout: 5000 });
console.log(JSON.stringify(res));

meckert er bei mir genauso das await an.

[ticaki]

@ben1983 sagte in auf globale Objekt kann nicht zugegriffen werden:

@ticaki ok und woran liegt es?
Es wird der Toplevel angeckert... die war aber schon länger so undmachte eigentlich keine Probleme.
Woran liegt jetzt der Fehler?
Habe ich was übersehen?

Bei mir stürzt die Javascript-Instanz ab, wenn ich deinen Code ausführe.

EDIT:
Du darfst await nur innerhalb einer Funktion benutzen, die mit async markiert ist.

[Ben1983]

@ticaki ja bei mir auch, aber warum?