profxadke/misskey
1
0
Fork 0
Code Issues Pull requests Projects Releases Packages Wiki Activity

Improve validation error message

Browse source
This commit is contained in:
mei23 2018-08-31 16:46:24 +09:00
parent e53dde385c
commit 18b4f74cdb
1 changed files with 6 additions and 3 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

9
src/queue/processors/http/process-inbox.ts
View file

@ -38,7 +38,7 @@ export default async (job: bq.Job, done: any): Promise<void> => {
try { try {
ValidateActivity(activity, host); ValidateActivity(activity, host);
} catch (e) { } catch (e) {
console.warn(e); console.warn(e.message);
done(); done();
return; return;
} }
@ -55,7 +55,7 @@ export default async (job: bq.Job, done: any): Promise<void> => {
try { try {
ValidateActivity(activity, host); ValidateActivity(activity, host);
} catch (e) { } catch (e) {
console.warn(e); console.warn(e.message);
done(); done();
return; return;
} }
@ -100,7 +100,10 @@ function ValidateActivity(activity: any, host: string) {
// id (if exists) // id (if exists)
if (typeof activity.id === 'string') { if (typeof activity.id === 'string') {
const uriHost = toUnicode(new URL(activity.id).hostname.toLowerCase()); const uriHost = toUnicode(new URL(activity.id).hostname.toLowerCase());
if (host !== uriHost) throw new Error('activity.id has different host'); if (host !== uriHost) {
const diag = activity.signature ? '. Has LD-Signature. Forwarded?' : '';
throw new Error(`activity.id(${activity.id}) has different host(${host})${diag}`);
}
} }
// actor (if exists) // actor (if exists)