Don't render opengraph tags of private videos
authorChocobozzz <me@florianbigard.com>
Thu, 24 May 2018 13:34:16 +0000 (15:34 +0200)
committerChocobozzz <me@florianbigard.com>
Thu, 24 May 2018 13:34:16 +0000 (15:34 +0200)
server/controllers/client.ts

index 20f7e5c9c2a613abee62adb73310615c11f991b5..aff00fe6e7b4bbbe7ba211792ddc6a3caf910e59 100644 (file)
@@ -6,6 +6,7 @@ import { escapeHTML, readFileBufferPromise, root } from '../helpers/core-utils'
 import { CONFIG, EMBED_SIZE, OPENGRAPH_AND_OEMBED_COMMENT, STATIC_MAX_AGE, STATIC_PATHS } from '../initializers'
 import { asyncMiddleware } from '../middlewares'
 import { VideoModel } from '../models/video/video'
+import { VideoPrivacy } from '../../shared/models/videos'
 
 const clientsRouter = express.Router()
 
@@ -152,7 +153,7 @@ async function generateWatchHtmlPage (req: express.Request, res: express.Respons
   const html = file.toString()
 
   // Let Angular application handle errors
-  if (!video) return res.sendFile(indexPath)
+  if (!video || video.privacy === VideoPrivacy.PRIVATE) return res.sendFile(indexPath)
 
   const htmlStringPageWithTags = addOpenGraphAndOEmbedTags(html, video)
   res.set('Content-Type', 'text/html; charset=UTF-8').send(htmlStringPageWithTags)