Add search target check params

[github/Chocobozzz/PeerTube.git] / server / lib / client-html.ts
diff --git a/server/lib/client-html.ts b/server/lib/client-html.ts

index e5950561445015452d4d0ea8e7dc45bdcda5aa87..4a4b0d12f9799b1d21455a0cafb16e925e625504 100644 (file)
--- a/server/lib/client-html.ts
+++ b/server/lib/client-html.ts
@@ -17,7 +17,7 @@ import { MAccountActor, MChannelActor, MVideo } from '../typings/models'
  
  export class ClientHtml {
  
-  private static htmlCache: { [ path: string ]: string } = {}
+  private static htmlCache: { [path: string]: string } = {}
  
    static invalidCache () {
      logger.info('Cleaning HTML cache.')
@@ -26,7 +26,9 @@ export class ClientHtml {
    }
  
    static async getDefaultHTMLPage (req: express.Request, res: express.Response, paramLang?: string) {
-    const html = await ClientHtml.getIndexHTML(req, res, paramLang)
+    const html = paramLang
+      ? await ClientHtml.getIndexHTML(req, res, paramLang)
+      : await ClientHtml.getIndexHTML(req, res)
  
      let customHtml = ClientHtml.addTitleTag(html)
      customHtml = ClientHtml.addDescriptionTag(customHtml)
@@ -92,21 +94,22 @@ export class ClientHtml {
  
    private static async getIndexHTML (req: express.Request, res: express.Response, paramLang?: string) {
      const path = ClientHtml.getIndexPath(req, res, paramLang)
-    if (ClientHtml.htmlCache[ path ]) return ClientHtml.htmlCache[ path ]
+    if (ClientHtml.htmlCache[path]) return ClientHtml.htmlCache[path]
  
      const buffer = await readFile(path)
  
      let html = buffer.toString()
  
+    if (paramLang) html = ClientHtml.addHtmlLang(html, paramLang)
      html = ClientHtml.addCustomCSS(html)
      html = await ClientHtml.addAsyncPluginCSS(html)
  
-    ClientHtml.htmlCache[ path ] = html
+    ClientHtml.htmlCache[path] = html
  
      return html
    }
  
-  private static getIndexPath (req: express.Request, res: express.Response, paramLang?: string) {
+  private static getIndexPath (req: express.Request, res: express.Response, paramLang: string) {
      let lang: string
  
      // Check param lang validity
@@ -116,7 +119,7 @@ export class ClientHtml {
        // Save locale in cookies
        res.cookie('clientLanguage', lang, {
          secure: WEBSERVER.SCHEME === 'https',
-        sameSite: true,
+        sameSite: 'none',
          maxAge: 1000 * 3600 * 24 * 90 // 3 months
        })
  
@@ -129,6 +132,10 @@ export class ClientHtml {
      return join(__dirname, '../../../client/dist/' + buildFileLocale(lang) + '/index.html')
    }
  
+  private static addHtmlLang (htmlStringPage: string, paramLang: string) {
+    return htmlStringPage.replace('<html>', `<html lang="${paramLang}">`)
+  }
+
    private static addTitleTag (htmlStringPage: string, title?: string) {
      let text = title || CONFIG.INSTANCE.NAME
      if (title) text += ` - ${CONFIG.INSTANCE.NAME}`
@@ -207,21 +214,21 @@ export class ClientHtml {
      const schemaTags = {
        '@context': 'http://schema.org',
        '@type': 'VideoObject',
-      name: videoNameEscaped,
-      description: videoDescriptionEscaped,
-      thumbnailUrl: previewUrl,
-      uploadDate: video.createdAt.toISOString(),
-      duration: getActivityStreamDuration(video.duration),
-      contentUrl: videoUrl,
-      embedUrl: embedUrl,
-      interactionCount: video.views
+      'name': videoNameEscaped,
+      'description': videoDescriptionEscaped,
+      'thumbnailUrl': previewUrl,
+      'uploadDate': video.createdAt.toISOString(),
+      'duration': getActivityStreamDuration(video.duration),
+      'contentUrl': videoUrl,
+      'embedUrl': embedUrl,
+      'interactionCount': video.views
      }
  
      let tagsString = ''
  
      // Opengraph
      Object.keys(openGraphMetaTags).forEach(tagName => {
-      const tagValue = openGraphMetaTags[ tagName ]
+      const tagValue = openGraphMetaTags[tagName]
  
        tagsString += `<meta property="${tagName}" content="${tagValue}" />`
      })