Nemcio/NodeBB
1
0
Fork 0
mirror of https://github.com/NodeBB/NodeBB.git synced 2026-01-05 07:10:30 +01:00
Code Issues Packages Projects Releases Wiki Activity

fix: remove unnecessary check and guard against unsupported URIs before fetching

Browse Source
This commit is contained in:
Opliko
2024-05-06 23:57:47 +02:00
parent 22b42f11dd
commit af98675808
1 changed files with 4 additions and 2 deletions
Download Patch File Download Diff File Expand all files Collapse all files

6
src/activitypub/actors.js
View File

@@ -33,7 +33,7 @@ Actors.assert = async (ids, options = {}) => {
const originalId = id;
const isUri = activitypub.helpers.isUri(id);
// only look up webfinger if the id is not a supported URI
if (id.includes('@') && !(isUri && activitypub._constants.acceptedProtocols.includes(new URL(id).protocol.slice(0, -1)))) {
if (id.includes('@') && !isUri) {
const host = isUri ? new URL(id).host : id.split('@')[1];
if (host === nconf.get('url_parsed').host) { // do not assert loopback ids
return 'loopback';
@@ -41,8 +41,10 @@ Actors.assert = async (ids, options = {}) => {
({ actorUri: id } = await activitypub.helpers.query(id));
}
if (!id) {
// ensure the final id is a valid URI
if (!id || !activitypub.helpers.isUri(id)) {
failedWebfingerCache.set(originalId, true);
return;
}
return id;
})));