currentTime is now required

1 files changed, 2 insertions, 9 deletions

diff --git a/server/middlewares/validators/videos/video-view.ts b/server/middlewares/validators/videos/video-view.ts
index 6e2d4505d..a2f61f4ba 100644
--- a/server/middlewares/validators/videos/video-view.ts
+++ b/server/middlewares/validators/videos/video-view.ts
@@ -4,7 +4,7 @@ import { isVideoTimeValid } from '@server/helpers/custom-validators/video-view'
 import { getCachedVideoDuration } from '@server/lib/video'
 import { LocalVideoViewerModel } from '@server/models/view/local-video-viewer'
 import { HttpStatusCode } from '../../../../shared/models/http/http-error-codes'
-import { exists, isIdValid, isIntOrNull, toIntOrNull } from '../../../helpers/custom-validators/misc'
+import { isIdValid, isIntOrNull, toIntOrNull } from '../../../helpers/custom-validators/misc'
 import { areValidationErrors, doesVideoExist, isValidVideoIdParam } from '../shared'
 
 const getVideoLocalViewerValidator = [
@@ -32,7 +32,6 @@ const videoViewValidator = [
   isValidVideoIdParam('videoId'),
 
   body('currentTime')
-    .optional() // TODO: remove optional in a few versions, introduced in 4.2
     .customSanitizer(toIntOrNull)
     .custom(isIntOrNull),
 
@@ -43,13 +42,7 @@ const videoViewValidator = [
     const video = res.locals.onlyImmutableVideo
     const { duration } = await getCachedVideoDuration(video.id)
 
-    if (!exists(req.body.currentTime)) { // TODO: remove in a few versions, introduced in 4.2
-      req.body.currentTime = Math.min(duration ?? 0, 30)
-    }
-
-    const currentTime: number = req.body.currentTime
-
-    if (!isVideoTimeValid(currentTime, duration)) {
+    if (!isVideoTimeValid(req.body.currentTime, duration)) {
       return res.fail({
         status: HttpStatusCode.BAD_REQUEST_400,
         message: 'Current time is invalid'