Mercurial > prosody-modules
comparison mod_rest/mod_rest.lua @ 4249:64aa1d9d70ac
mod_rest: Catch and log errors in callback promise chain
From the code it looks like it should be possible to reply to an error
stanza, but it did not. Turns out I was saved by my local developer mode
module which throws errors if an attempt is made to create an errror
reply to an error stanza. However nothing collects this error from the
promise, so all I got was confusion.
| author | Kim Alvefur <zash@zash.se> |
|---|---|
| date | Sun, 15 Nov 2020 16:25:49 +0100 |
| parents | 1f93fa24611d |
| children | 8b489203e4d3 |
comparison
equal
deleted
inserted
replaced
| 4248:f3fbfde9683d | 4249:64aa1d9d70ac |
|---|---|
| 390 origin.send(reply); | 390 origin.send(reply); |
| 391 end, | 391 end, |
| 392 function (err) | 392 function (err) |
| 393 module:log_status("error", "Could not connect to callback URL %q: %s", rest_url, err); | 393 module:log_status("error", "Could not connect to callback URL %q: %s", rest_url, err); |
| 394 origin.send(st.error_reply(stanza, "wait", "recipient-unavailable", err.text)); | 394 origin.send(st.error_reply(stanza, "wait", "recipient-unavailable", err.text)); |
| 395 end):catch(function (err) | |
| 396 module:log("error", "Error[rest]: %s", err); | |
| 395 end); | 397 end); |
| 396 | 398 |
| 397 return true; | 399 return true; |
| 398 end | 400 end |
| 399 | 401 |