X-Git-Url: https://git.immae.eu/?a=blobdiff_plain;f=server%2Fmiddlewares%2Fvalidators%2Factivitypub%2Factivity.ts;h=d24e4427b79cf5191ca4dd364fc72c004ed7e06a;hb=c4fa01f7c45b66b112ebd08abce744b7c4041feb;hp=c63be5979053cec022d2f594db54b2f6bd7b569c;hpb=fef2c7164e025b12a64185dbab058ef4129733c6;p=github%2FChocobozzz%2FPeerTube.git

diff --git a/server/middlewares/validators/activitypub/activity.ts b/server/middlewares/validators/activitypub/activity.ts
index c63be5979..d24e4427b 100644
--- a/server/middlewares/validators/activitypub/activity.ts
+++ b/server/middlewares/validators/activitypub/activity.ts
@@ -1,19 +1,26 @@
 import * as express from 'express'
-import { body } from 'express-validator/check'
-import { isRootActivityValid, logger } from '../../../helpers'
-import { areValidationErrors } from '../utils'
+import { getServerActor } from '@server/models/application/application'
+import { HttpStatusCode } from '../../../../shared/models/http/http-error-codes'
+import { isRootActivityValid } from '../../../helpers/custom-validators/activitypub/activity'
+import { logger } from '../../../helpers/logger'
 
-const activityPubValidator = [
-  body('').custom((value, { req }) => isRootActivityValid(req.body)),
+async function activityPubValidator (req: express.Request, res: express.Response, next: express.NextFunction) {
+  logger.debug('Checking activity pub parameters')
 
-  (req: express.Request, res: express.Response, next: express.NextFunction) => {
-    logger.debug('Checking activity pub parameters')
-
-    if (areValidationErrors(req, res)) return
+  if (!isRootActivityValid(req.body)) {
+    logger.warn('Incorrect activity parameters.', { activity: req.body })
+    return res.fail({ message: 'Incorrect activity' })
+  }
 
-    return next()
+  const serverActor = await getServerActor()
+  const remoteActor = res.locals.signature.actor
+  if (serverActor.id === remoteActor.id || remoteActor.serverId === null) {
+    logger.error('Receiving request in INBOX by ourselves!', req.body)
+    return res.status(HttpStatusCode.CONFLICT_409).end()
   }
-]
+
+  return next()
+}
 
 // ---------------------------------------------------------------------------