X-Git-Url: https://git.immae.eu/?a=blobdiff_plain;f=server%2Fcontrollers%2Factivitypub%2Finbox.ts;h=30e7f706b95448bf45d2ad012625d22c3b1f6775;hb=d52eb8f656242c7e34afdb2dee681861fb9bce35;hp=79d989c2c26d6d0bb1e55477f2947fb151c23843;hpb=e4f97babf701481b55cc10fb3448feab5f97c867;p=github%2FChocobozzz%2FPeerTube.git diff --git a/server/controllers/activitypub/inbox.ts b/server/controllers/activitypub/inbox.ts index 79d989c2c..30e7f706b 100644 --- a/server/controllers/activitypub/inbox.ts +++ b/server/controllers/activitypub/inbox.ts @@ -1,36 +1,25 @@ import * as express from 'express' - -import { - processCreateActivity, - processUpdateActivity, - processFlagActivity -} from '../../lib' -import { - Activity, - ActivityType, - RootActivity, - ActivityPubCollection, - ActivityPubOrderedCollection -} from '../../../shared' -import { - signatureValidator, - checkSignature, - asyncMiddleware -} from '../../middlewares' +import { Activity, ActivityPubCollection, ActivityPubOrderedCollection, RootActivity } from '../../../shared' import { logger } from '../../helpers' - -const processActivity: { [ P in ActivityType ]: (activity: Activity) => Promise } = { - Create: processCreateActivity, - Update: processUpdateActivity, - Flag: processFlagActivity -} +import { isActivityValid } from '../../helpers/custom-validators/activitypub/activity' +import { processActivities } from '../../lib/activitypub/process/process' +import { asyncMiddleware, checkSignature, localAccountValidator, signatureValidator } from '../../middlewares' +import { activityPubValidator } from '../../middlewares/validators/activitypub/activity' const inboxRouter = express.Router() -inboxRouter.post('/', +inboxRouter.post('/inbox', + signatureValidator, + asyncMiddleware(checkSignature), + activityPubValidator, + asyncMiddleware(inboxController) +) + +inboxRouter.post('/account/:name/inbox', signatureValidator, asyncMiddleware(checkSignature), - // inboxValidator, + localAccountValidator, + activityPubValidator, asyncMiddleware(inboxController) ) @@ -54,19 +43,12 @@ async function inboxController (req: express.Request, res: express.Response, nex activities = [ rootActivity as Activity ] } - await processActivities(activities) + // Only keep activities we are able to process + logger.debug('Filtering activities...', { activities }) + activities = activities.filter(a => isActivityValid(a)) + logger.debug('We keep %d activities.', activities.length, { activities }) - res.status(204).end() -} + await processActivities(activities, res.locals.account) -async function processActivities (activities: Activity[]) { - for (const activity of activities) { - const activityProcessor = processActivity[activity.type] - if (activityProcessor === undefined) { - logger.warn('Unknown activity type %s.', activity.type, { activityId: activity.id }) - continue - } - - await activityProcessor(activity) - } + res.status(204).end() }