Fix bug

src/remote/activitypub/models/person.ts

@@ -47,16 +47,28 @@ export async function createPerson(value: any, resolver?: Resolver): Promise<IUs
 
 	const object = await resolver.resolve(value) as any;
 
-	if (
+	if (object == null) {
-		object == null ||
+		throw new Error('invalid person: object is null');
-		object.type !== 'Person' ||
+	}
-		typeof object.preferredUsername !== 'string' ||
+
-		typeof object.inbox !== 'string' ||
+	if (object.type != 'Person' && object.type != 'Service') {
-		!validateUsername(object.preferredUsername) ||
+		throw new Error('invalid person: object is not a person or service');
-		!isValidName(object.name == '' ? null : object.name)
+	}
-	) {
+
-		log(`invalid person: ${JSON.stringify(object, null, 2)}`);
+	if (typeof object.preferredUsername !== 'string') {
-		throw new Error('invalid person');
+		throw new Error('invalid person: preferredUsername is not a string');
+	}
+
+	if (typeof object.inbox !== 'string') {
+		throw new Error('invalid person: inbox is not a string');
+	}
+
+	if (!validateUsername(object.preferredUsername)) {
+		throw new Error('invalid person: invalid username');
+	}
+
+	if (!isValidName(object.name == '' ? null : object.name)) {
+		throw new Error('invalid person: invalid name');
 	}
 
 	const person: IPerson = object;

src/remote/activitypub/renderer/person.ts

@@ -7,7 +7,7 @@ export default (user: ILocalUser) => {
 	const id = `${config.url}/users/${user._id}`;
 
 	return {
-		type: 'Person',
+		type: user.isBot ? 'Service' : 'Person',
 		id,
 		inbox: `${id}/inbox`,
 		outbox: `${id}/outbox`,