Merge pull request #255 from tinas/fix-case-insensitive-content-type

fix: normalize content-type header check for case insensitivity

Author: jshemas

lib/request.ts

@@ -72,7 +72,8 @@ export default async function requestAndResultsFormatter(options: OpenGraphScrap
       body = decode(Buffer.from(bodyArrayBuffer), charset);
     }
 
-    if (response?.headers?.get('content-type') && !response.headers.get('content-type')?.includes('text/')) {
+    const contentType = response?.headers?.get('content-type')?.toLowerCase();
+    if (contentType && !contentType.includes('text/')) {
       throw new Error('Page must return a header content-type with text/');
     }
     if (response?.status && (response.status.toString().startsWith('4') || response.status.toString().startsWith('5'))) {

tests/unit/openGraphScraper.spec.ts

@@ -709,6 +709,22 @@ describe('return ogs', function () {
         });
     });
 
+    it('should handle case-insensitive Content-Type headers', function () {
+      mockAgent.get('http://www.test.com')
+        .intercept({ path: '/' })
+        .reply(200, basicHTML, { headers: { 'Content-Type': 'Text/html; charset=utf-8' } });
+
+      return ogs({ url: 'http://www.test.com' })
+        .then(function (data) {
+          expect(data.result.success).to.be.eql(true);
+          expect(data.error).to.be.eql(false);
+          expect(data.result.ogTitle).to.be.eql('test page');
+          expect(data.result.requestUrl).to.be.eql('http://www.test.com');
+          expect(data.html).to.be.eql(basicHTML);
+          expect(data.response).to.be.a('response');
+        });
+    });
+
     it('when trying to hit a non html pages based on content-type', function () {
       mockAgent.get('http://www.test.com')
         .intercept({ path: '/' })