Skip to content
This repository was archived by the owner on May 21, 2025. It is now read-only.

Commit 2665e6d

Browse files
authored
Merge pull request #379 from autonomoussoftware/fix-send-error
Fix sending error messages from main to renderer
2 parents 904b000 + 7d6ba52 commit 2665e6d

File tree

1 file changed

+9
-0
lines changed

1 file changed

+9
-0
lines changed

public/main/main-worker.js

Lines changed: 9 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -25,13 +25,22 @@ function onRendererEvent (eventName, listener) {
2525
const result = Promise.resolve(listener(data, event.sender))
2626

2727
result
28+
.then(res => res.error ? Promise.reject(res.error) : res)
2829
.then(function (res) {
2930
if (event.sender.isDestroyed()) {
3031
return
3132
}
3233
event.sender.send(eventName, { id, data: res })
3334
logger.verbose(`<-- ${eventName}:${id} ${JSON.stringify(res)}`)
3435
})
36+
.catch(function (err) {
37+
if (event.sender.isDestroyed()) {
38+
return
39+
}
40+
const error = new WalletError(err.message)
41+
event.sender.send(eventName, { id, data: { error } })
42+
logger.warn(`<-- ${eventName}:${id} ${err.message}`)
43+
})
3544
.catch(function (err) {
3645
logger.warn(`Could not send message to renderer: ${err.message}`)
3746
})

0 commit comments

Comments
 (0)