Support empty value returned by video get filter

packages/tests/src/api/videos/video-source.ts

@@ -317,7 +317,7 @@ describe('Test a video file replacement', function () {
       })
 
       it('Should not auto blacklist if auto blacklist has been disabled between the upload and the replacement', async function () {
-        this.timeout(120000)
+        this.timeout(240000)
 
         const { uuid } = await servers[0].videos.quickUpload({ token: userToken, name: 'user video' })
         await waitJobs(servers)

server/server/controllers/api/videos/index.ts

@@ -155,6 +155,8 @@ async function getVideo (_req: express.Request, res: express.Response) {
   const userId = res.locals.oauth?.token.User.id
 
   const video = await Hooks.wrapObject(res.locals.videoAPI, 'filter:api.video.get.result', { id: videoId, userId })
+  // Filter may return null/undefined value to forbid video access
+  if (!video) return res.sendStatus(HttpStatusCode.NOT_FOUND_404)
 
   if (video.isOutdated()) {
     JobQueue.Instance.createJobAsync({ type: 'activitypub-refresher', payload: { type: 'video', url: video.url } })

server/server/lib/job-queue/handlers/video-live-ending.ts

@@ -263,8 +263,6 @@ async function assignReplayFilesToVideo (options: {
         fps,
         isAAC: audioStream?.codec_name === 'aac'
       })
-
-      logger.error('coucou')
     } catch (err) {
       logger.error('Cannot generate HLS playlist resolution from TS files.', { err })
     }