Add contentType check to XHR documentLoader link header test.

gkellogg · gkellogg · commit f8db5b457d21 · 2020-01-28T16:59:13.000-08:00
diff --git a/lib/documentLoaders/xhr.js b/lib/documentLoaders/xhr.js
@@ -89,7 +89,8 @@ module.exports = ({
 
     // handle Link Header (avoid unsafe header warning by existence testing)
     let linkHeader;
-    if(REGEX_LINK_HEADER.test(req.getAllResponseHeaders())) {
+    if(REGEX_LINK_HEADER.test(req.getAllResponseHeaders()) &&
+      contentType !== 'application/ld+json') {
       linkHeader = req.getResponseHeader('Link');
     }
     if(linkHeader && contentType !== 'application/ld+json') {

Original file line number	Diff line number	Diff line change
`@@ -89,7 +89,8 @@ module.exports = ({`
`89`	`89`
`90`	`90`	`// handle Link Header (avoid unsafe header warning by existence testing)`
`91`	`91`	`let linkHeader;`
`92`		`- if(REGEX_LINK_HEADER.test(req.getAllResponseHeaders())) {`
	`92`	`+ if(REGEX_LINK_HEADER.test(req.getAllResponseHeaders()) &&`
	`93`	`+ contentType !== 'application/ld+json') {`
`93`	`94`	`linkHeader = req.getResponseHeader('Link');`
`94`	`95`	`}`
`95`	`96`	`if(linkHeader && contentType !== 'application/ld+json') {`