some environments, throws when accessing document.cookie

Author: adrai

CHANGELOG.md

@@ -1,3 +1,7 @@
+### 8.0.1
+
+- some environments, throws when accessing document.cookie
+
 ### 8.0.0
 
 - chore: set browsers target to defaults [286](https://github.com/i18next/i18next-browser-languageDetector/pull/286)

LICENSE

@@ -1,6 +1,6 @@
 The MIT License (MIT)
 
-Copyright (c) 2023 i18next
+Copyright (c) 2024 i18next
 
 Permission is hereby granted, free of charge, to any person obtaining a copy
 of this software and associated documentation files (the "Software"), to deal

i18nextBrowserLanguageDetector.js

@@ -21,9 +21,11 @@
 
   // eslint-disable-next-line no-control-regex
   const fieldContentRegExp = /^[\u0009\u0020-\u007e\u0080-\u00ff]+$/;
-  const serializeCookie = (name, val, options) => {
-    const opt = options || {};
-    opt.path = opt.path || '/';
+  const serializeCookie = function (name, val) {
+    let options = arguments.length > 2 && arguments[2] !== undefined ? arguments[2] : {
+      path: '/'
+    };
+    const opt = options;
     const value = encodeURIComponent(val);
     let str = `${name}=${value}`;
     if (opt.maxAge > 0) {
@@ -179,7 +181,6 @@
       if (lookupLocalStorage && localStorageAvailable()) {
         return window.localStorage.getItem(lookupLocalStorage) || undefined; // Undefined ensures type consistency with the previous version of this function
       }
-
       return undefined;
     },
     // Deconstruct the options object and extract the lookupLocalStorage property
@@ -305,35 +306,43 @@
     }
   };
 
-  function getDefaults() {
-    return {
-      order: ['querystring', 'cookie', 'localStorage', 'sessionStorage', 'navigator', 'htmlTag'],
-      lookupQuerystring: 'lng',
-      lookupCookie: 'i18next',
-      lookupLocalStorage: 'i18nextLng',
-      lookupSessionStorage: 'i18nextLng',
-      // cache user language
-      caches: ['localStorage'],
-      excludeCacheFor: ['cimode'],
-      // cookieMinutes: 10,
-      // cookieDomain: 'myDomain'
+  let canCookies = false;
+  try {
+    // eslint-disable-next-line no-unused-expressions
+    document.cookie;
+    canCookies = true;
+    // eslint-disable-next-line no-empty
+  } catch (e) {}
+  const order = ['querystring', 'cookie', 'localStorage', 'sessionStorage', 'navigator', 'htmlTag'];
+  if (!canCookies) order.splice(1, 1);
+  const getDefaults = () => ({
+    order,
+    lookupQuerystring: 'lng',
+    lookupCookie: 'i18next',
+    lookupLocalStorage: 'i18nextLng',
+    lookupSessionStorage: 'i18nextLng',
+    // cache user language
+    caches: ['localStorage'],
+    excludeCacheFor: ['cimode'],
+    // cookieMinutes: 10,
+    // cookieDomain: 'myDomain'
 
-      convertDetectedLanguage: l => l
-    };
-  }
+    convertDetectedLanguage: l => l
+  });
   class Browser {
     constructor(services) {
       let options = arguments.length > 1 && arguments[1] !== undefined ? arguments[1] : {};
       this.type = 'languageDetector';
       this.detectors = {};
       this.init(services, options);
     }
-    init(services) {
+    init() {
+      let services = arguments.length > 0 && arguments[0] !== undefined ? arguments[0] : {
+        languageUtils: {}
+      };
       let options = arguments.length > 1 && arguments[1] !== undefined ? arguments[1] : {};
       let i18nOptions = arguments.length > 2 && arguments[2] !== undefined ? arguments[2] : {};
-      this.services = services || {
-        languageUtils: {}
-      }; // this way the language detector can be used without i18next
+      this.services = services;
       this.options = defaults(options, this.options || {}, getDefaults());
       if (typeof this.options.convertDetectedLanguage === 'string' && this.options.convertDetectedLanguage.indexOf('15897') > -1) {
         this.options.convertDetectedLanguage = l => l.replace('-', '_');
@@ -355,8 +364,8 @@
       this.detectors[detector.name] = detector;
       return this;
     }
-    detect(detectionOrder) {
-      if (!detectionOrder) detectionOrder = this.options.order;
+    detect() {
+      let detectionOrder = arguments.length > 0 && arguments[0] !== undefined ? arguments[0] : this.options.order;
       let detected = [];
       detectionOrder.forEach(detectorName => {
         if (this.detectors[detectorName]) {
@@ -369,9 +378,8 @@
       if (this.services.languageUtils.getBestMatchFromCodes) return detected; // new i18next v19.5.0
       return detected.length > 0 ? detected[0] : null; // a little backward compatibility
     }
-
-    cacheUserLanguage(lng, caches) {
-      if (!caches) caches = this.options.caches;
+    cacheUserLanguage(lng) {
+      let caches = arguments.length > 1 && arguments[1] !== undefined ? arguments[1] : this.options.caches;
       if (!caches) return;
       if (this.options.excludeCacheFor && this.options.excludeCacheFor.indexOf(lng) > -1) return;
       caches.forEach(cacheName => {

i18nextBrowserLanguageDetector.min.js

@@ -8,8 +8,18 @@ import htmlTag from './browserLookups/htmlTag.js';
 import path from './browserLookups/path.js';
 import subdomain from './browserLookups/subdomain.js';
 
+// some environments, throws when accessing document.cookie
+let canCookies = false;
+try {
+  // eslint-disable-next-line no-unused-expressions
+  document.cookie;
+  canCookies = true;
+// eslint-disable-next-line no-empty
+} catch (e) {}
+const order = ['querystring', 'cookie', 'localStorage', 'sessionStorage', 'navigator', 'htmlTag'];
+if (!canCookies) order.splice(1, 1);
 const getDefaults = () => ({
-  order: ['querystring', 'cookie', 'localStorage', 'sessionStorage', 'navigator', 'htmlTag'],
+  order,
   lookupQuerystring: 'lng',
   lookupCookie: 'i18next',
   lookupLocalStorage: 'i18nextLng',