Merge pull request #279 from ndaidong/7.0.0

v7.0.0rc3
extractus · Jul 11, 2022 · c6394e6 · c6394e6
2 parents a78e306 + df62e0b
commit c6394e6
Show file tree

Hide file tree

Showing 8 changed files with 17 additions and 15 deletions.
diff --git a/dist/article-parser.browser.js b/dist/article-parser.browser.js
diff --git a/dist/article-parser.browser.js.map b/dist/article-parser.browser.js.map
diff --git a/dist/cjs/article-parser.js b/dist/cjs/article-parser.js
diff --git a/dist/cjs/article-parser.js.map b/dist/cjs/article-parser.js.map
diff --git a/dist/cjs/package.json b/dist/cjs/package.json
@@ -1,5 +1,5 @@
 {
   "name": "article-parser-cjs",
-  "version": "7.0.0rc2",
+  "version": "7.0.0rc3",
   "main": "./article-parser.js"
 }
diff --git a/package.json b/package.json
@@ -1,5 +1,5 @@
 {
-  "version": "7.0.0rc2",
+  "version": "7.0.0rc3",
   "name": "article-parser",
   "description": "To extract main article from given URL",
   "homepage": "https://ndaidong.github.io/article-parser-demo/",
@@ -47,7 +47,7 @@
   "devDependencies": {
     "@types/sanitize-html": "^2.6.2",
     "cross-env": "^7.0.3",
-    "esbuild": "^0.14.48",
+    "esbuild": "^0.14.49",
     "jest": "^28.1.2",
     "nock": "^13.2.8",
     "standard": "^17.0.0"

diff --git a/src/config.js b/src/config.js
@@ -5,7 +5,8 @@ import { clone, copies } from 'bellajs'
 const requestOptions = {
   headers: {
     'user-agent': 'Mozilla/5.0 (X11; Linux x86_64; rv:102.0) Gecko/20100101 Firefox/102.0',
-    accept: 'text/html; charset=utf-8'
+    accept: 'text/html; charset=utf-8',
+    'accept-encoding': 'deflate,zlib,gzip'
   },
   responseType: 'text',
   responseEncoding: 'utf8',
@@ -21,10 +22,10 @@ const sanitizeHtmlOptions = {
     progress: ['value', 'max'],
     img: ['src', 'srcset', 'alt', 'width', 'height', 'style', 'title'],
     picture: ['media', 'srcset'],
-    video: ['controls', 'width', 'height', 'autoplay', 'muted'],
+    video: ['controls', 'width', 'height', 'autoplay', 'muted', 'loop', 'src'],
     audio: ['controls'],
     source: ['src', 'srcset', 'data-srcset', 'type', 'media', 'sizes'],
-    iframe: ['src', 'frameborder', 'height', 'width', 'scrolling'],
+    iframe: ['src', 'frameborder', 'height', 'width', 'scrolling', 'allow'],
     svg: ['width', 'height'] // sanitize-html does not support svg fully yet
   },
   allowedIframeDomains: ['youtube.com', 'twitter.com', 'facebook.com', 'vimeo.com']

diff --git a/src/config.test.js b/src/config.test.js
@@ -25,7 +25,8 @@ test('Testing setRequestOptions/getRequestOptions methods', () => {
   const expectedHeader = {
     authorization: 'bearer <token>',
     'user-agent': 'Mozilla/5.0 (X11; Linux x86_64; rv:102.0) Gecko/20100101 Firefox/102.0',
-    accept: 'text/html; charset=utf-8'
+    accept: 'text/html; charset=utf-8',
+    'accept-encoding': 'deflate,zlib,gzip'
   }
 
   expect(actual.headers).toEqual(expectedHeader)