From 0f4e9feafad9b616cde7276c6db204608be31900 Mon Sep 17 00:00:00 2001
From: Gary Meehan <garymeehan@outlook.com>
Date: Sun, 12 Aug 2018 20:51:17 +0100
Subject: [PATCH] Allow Duplicate Open Graph Image Tags (#4851)

* allow og:image duplicates

* add tests

* update tests

* fix spacing
---
 lib/head.js                              |  2 +-
 test/integration/basic/pages/head.js     | 14 ++++++++++++++
 test/integration/basic/test/rendering.js | 14 ++++++++++++++
 3 files changed, 29 insertions(+), 1 deletion(-)

diff --git a/lib/head.js b/lib/head.js
index 5afbb8f6..d7b1423f 100644
--- a/lib/head.js
+++ b/lib/head.js
@@ -49,7 +49,7 @@ function onStateChange (head) {
 }
 
 const METATYPES = ['name', 'httpEquiv', 'charSet', 'itemProp', 'property']
-const ALLOWED_DUPLICATES = ['article:tag']
+const ALLOWED_DUPLICATES = ['article:tag', 'og:image', 'og:image:alt', 'og:image:width', 'og:image:height', 'og:image:type', 'og:image:secure_url', 'og:image:url']
 
 /*
  returns a function for filtering head child elements
diff --git a/test/integration/basic/pages/head.js b/test/integration/basic/pages/head.js
index 05bc13e9..9e219a82 100644
--- a/test/integration/basic/pages/head.js
+++ b/test/integration/basic/pages/head.js
@@ -15,6 +15,20 @@ export default () => <div>
     <meta property='article:tag' content='tag2' key='tag2key' />
     <meta property='dedupe:tag' content='tag3' key='tag3key' />
     <meta property='dedupe:tag' content='tag4' key='tag4key' />
+    <meta property='og:image' content='ogImageTag1' key='ogImageTag1Key' />
+    <meta property='og:image' content='ogImageTag2' key='ogImageTag2Key' />
+    <meta property='og:image:alt' content='ogImageAltTag1' key='ogImageAltTag1Key' />
+    <meta property='og:image:alt' content='ogImageAltTag2' key='ogImageAltTag2Key' />
+    <meta property='og:image:width' content='ogImageWidthTag1' key='ogImageWidthTag1Key' />
+    <meta property='og:image:width' content='ogImageWidthTag2' key='ogImageWidthTag2Key' />
+    <meta property='og:image:height' content='ogImageHeightTag1' key='ogImageHeightTag1Key' />
+    <meta property='og:image:height' content='ogImageHeightTag2' key='ogImageHeightTag2Key' />
+    <meta property='og:image:type' content='ogImageTypeTag1' key='ogImageTypeTag1Key' />
+    <meta property='og:image:type' content='ogImageTypeTag2' key='ogImageTypeTag2Key' />
+    <meta property='og:image:secure_url' content='ogImageSecureUrlTag1' key='ogImageSecureUrlTag1Key' />
+    <meta property='og:image:secure_url' content='ogImageSecureUrlTag2' key='ogImageSecureUrlTag2Key' />
+    <meta property='og:image:url' content='ogImageUrlTag1' key='ogImageUrlTag1Key' />
+    <meta property='og:image:url' content='ogImageUrlTag2' key='ogImageUrlTag2Key' />
 
     <React.Fragment>
       <title>Fragment title</title>
diff --git a/test/integration/basic/test/rendering.js b/test/integration/basic/test/rendering.js
index 16f3ac15..3f375edc 100644
--- a/test/integration/basic/test/rendering.js
+++ b/test/integration/basic/test/rendering.js
@@ -49,6 +49,20 @@ export default function ({ app }, suiteName, render, fetch) {
       expect(html).toContain('<meta property="article:tag" content="tag2" class="next-head"/>')
       expect(html).not.toContain('<meta property="dedupe:tag" content="tag3" class="next-head"/>')
       expect(html).toContain('<meta property="dedupe:tag" content="tag4" class="next-head"/>')
+      expect(html).toContain('<meta property="og:image" content="ogImageTag1" class="next-head"/>')
+      expect(html).toContain('<meta property="og:image" content="ogImageTag2" class="next-head"/>')
+      expect(html).toContain('<meta property="og:image:alt" content="ogImageAltTag1" class="next-head"/>')
+      expect(html).toContain('<meta property="og:image:alt" content="ogImageAltTag2" class="next-head"/>')
+      expect(html).toContain('<meta property="og:image:width" content="ogImageWidthTag1" class="next-head"/>')
+      expect(html).toContain('<meta property="og:image:width" content="ogImageWidthTag2" class="next-head"/>')
+      expect(html).toContain('<meta property="og:image:height" content="ogImageHeightTag1" class="next-head"/>')
+      expect(html).toContain('<meta property="og:image:height" content="ogImageHeightTag2" class="next-head"/>')
+      expect(html).toContain('<meta property="og:image:type" content="ogImageTypeTag1" class="next-head"/>')
+      expect(html).toContain('<meta property="og:image:type" content="ogImageTypeTag2" class="next-head"/>')
+      expect(html).toContain('<meta property="og:image:secure_url" content="ogImageSecureUrlTag1" class="next-head"/>')
+      expect(html).toContain('<meta property="og:image:secure_url" content="ogImageSecureUrlTag2" class="next-head"/>')
+      expect(html).toContain('<meta property="og:image:url" content="ogImageUrlTag1" class="next-head"/>')
+      expect(html).toContain('<meta property="og:image:url" content="ogImageUrlTag2" class="next-head"/>')
     })
 
     test('header helper renders Fragment children', async () => {