Nemcio/NodeBB
1
0
Fork 0
mirror of https://github.com/NodeBB/NodeBB.git synced 2025-10-26 16:46:12 +01:00
Code Issues Packages Projects Releases Wiki Activity

fix: wrap try..catch around topics.reply in note assertion to output errors

Browse Source
This commit is contained in:
Julian Lam
2024-12-11 14:40:21 -05:00
parent 8ec388afca
commit b551abce59
1 changed files with 11 additions and 7 deletions
Download Patch File Download Diff File Expand all files Collapse all files

18
src/activitypub/notes.js
View File

@@ -196,13 +196,17 @@ Notes.assert = async (uid, input, options = { skipChecks: false }) => {
for (const post of unprocessed) { for (const post of unprocessed) {
const { to, cc, attachment } = post._activitypub; const { to, cc, attachment } = post._activitypub;
// eslint-disable-next-line no-await-in-loop try {
await topics.reply(post); // eslint-disable-next-line no-await-in-loop
// eslint-disable-next-line no-await-in-loop await topics.reply(post);
await Promise.all([ // eslint-disable-next-line no-await-in-loop
Notes.updateLocalRecipients(post.pid, { to, cc }), await Promise.all([
posts.attachments.update(post.pid, attachment), Notes.updateLocalRecipients(post.pid, { to, cc }),
]); posts.attachments.update(post.pid, attachment),
]);
} catch (e) {
activitypub.helpers.log(`[activitypub/notes.assert] Could not add reply (${post.pid}): ${e.message}`);
}
} }
await Promise.all([ await Promise.all([