Add correct status on not found video
authorChocobozzz <me@florianbigard.com>
Thu, 9 Jan 2020 07:50:00 +0000 (08:50 +0100)
committerChocobozzz <me@florianbigard.com>
Thu, 9 Jan 2020 07:51:27 +0000 (08:51 +0100)
server/lib/client-html.ts

index 2fcf0c4eae2357e87a79f46edd478ea26f843dad..e5950561445015452d4d0ea8e7dc45bdcda5aa87 100644 (file)
@@ -37,6 +37,7 @@ export class ClientHtml {
   static async getWatchHTMLPage (videoId: string, req: express.Request, res: express.Response) {
     // Let Angular application handle errors
     if (!validator.isInt(videoId) && !validator.isUUID(videoId, 4)) {
+      res.status(404)
       return ClientHtml.getIndexHTML(req, res)
     }
 
@@ -47,7 +48,8 @@ export class ClientHtml {
 
     // Let Angular application handle errors
     if (!video || video.privacy === VideoPrivacy.PRIVATE || video.privacy === VideoPrivacy.INTERNAL || video.VideoBlacklist) {
-      return ClientHtml.getIndexHTML(req, res)
+      res.status(404)
+      return html
     }
 
     let customHtml = ClientHtml.addTitleTag(html, escapeHTML(video.name))
@@ -77,6 +79,7 @@ export class ClientHtml {
 
     // Let Angular application handle errors
     if (!entity) {
+      res.status(404)
       return ClientHtml.getIndexHTML(req, res)
     }