i18next-http-middleware 3.10.1 → 3.10.2

package/README.md

@@ -270,6 +270,9 @@ const app = new App()
 
 ```js
 // missing keys make sure the body is parsed (i.e. with [body-parser](https://github.com/expressjs/body-parser#bodyparserjsonoptions))
+// this route takes writes from anyone who can reach it: every key in the body goes to the
+// backend's create (i18next-fs-backend writes it into a file) and stays in the server's memory,
+// so mount it only in development or behind authentication
 app.post('/locales/add/:lng/:ns', middleware.missingKeyHandler(i18next))
 // addPath for client: http://localhost:8080/locales/add/{{lng}}/{{ns}}
 

package/cjs/index.js

@@ -301,6 +301,8 @@ function missingKeyHandler(i18next) {
     var keySeparator = i18next.options && i18next.options.keySeparator;
     var utilsOfRequest = req.i18n && req.i18n.services && req.i18n.services.utils;
     var hasLoadedNamespace = utilsOfRequest && utilsOfRequest.hasLoadedNamespace;
+    var sharedNs = i18next.options && i18next.options.ns;
+    var nsWasKnown = !Array.isArray(sharedNs) || sharedNs.indexOf(ns) > -1;
     var saveMissingKeys = function saveMissingKeys(src) {
       if (!src || _typeof(src) !== 'object') return;
       for (var _i = 0, _Object$keys = Object.keys(src); _i < _Object$keys.length; _i++) {
@@ -308,6 +310,7 @@ function missingKeyHandler(i18next) {
         if (utils.hasUnsafeKeySegment(m, keySeparator)) continue;
         i18next.services.backendConnector.saveMissing([lng], ns, m, src[m], undefined, undefined, undefined, hasLoadedNamespace);
       }
+      if (!nsWasKnown && sharedNs.indexOf(ns) > -1) sharedNs.splice(sharedNs.indexOf(ns), 1);
     };
     if (typeof body === 'function') {
       var promise = body();

package/esm/index.js

@@ -284,6 +284,8 @@ export function missingKeyHandler(i18next) {
     var keySeparator = i18next.options && i18next.options.keySeparator;
     var utilsOfRequest = req.i18n && req.i18n.services && req.i18n.services.utils;
     var hasLoadedNamespace = utilsOfRequest && utilsOfRequest.hasLoadedNamespace;
+    var sharedNs = i18next.options && i18next.options.ns;
+    var nsWasKnown = !Array.isArray(sharedNs) || sharedNs.indexOf(ns) > -1;
     var saveMissingKeys = function saveMissingKeys(src) {
       if (!src || _typeof(src) !== 'object') return;
       for (var _i = 0, _Object$keys = Object.keys(src); _i < _Object$keys.length; _i++) {
@@ -291,6 +293,7 @@ export function missingKeyHandler(i18next) {
         if (utils.hasUnsafeKeySegment(m, keySeparator)) continue;
         i18next.services.backendConnector.saveMissing([lng], ns, m, src[m], undefined, undefined, undefined, hasLoadedNamespace);
       }
+      if (!nsWasKnown && sharedNs.indexOf(ns) > -1) sharedNs.splice(sharedNs.indexOf(ns), 1);
     };
     if (typeof body === 'function') {
       var promise = body();

package/lib/index.js

@@ -335,6 +335,11 @@ export function missingKeyHandler (i18next, options = {}) {
     const keySeparator = i18next.options && i18next.options.keySeparator
     const utilsOfRequest = req.i18n && req.i18n.services && req.i18n.services.utils
     const hasLoadedNamespace = utilsOfRequest && utilsOfRequest.hasLoadedNamespace
+    // saveMissing() writes the key to the store, which also adds its namespace to the
+    // shared i18next.options.ns: every request walks that list, so a namespace that
+    // came from this request must not stay in it
+    const sharedNs = i18next.options && i18next.options.ns
+    const nsWasKnown = !Array.isArray(sharedNs) || sharedNs.indexOf(ns) > -1
 
     // iterate only over own, non-prototype-polluting keys. The check also
     // rejects dotted variants like `__proto__.polluted` whose segments under
@@ -347,6 +352,7 @@ export function missingKeyHandler (i18next, options = {}) {
         // request) as the 8th argument; without it, it checks the newest clone
         i18next.services.backendConnector.saveMissing([lng], ns, m, src[m], undefined, undefined, undefined, hasLoadedNamespace)
       }
+      if (!nsWasKnown && sharedNs.indexOf(ns) > -1) sharedNs.splice(sharedNs.indexOf(ns), 1)
     }
 
     if (typeof body === 'function') {

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "i18next-http-middleware",
-  "version": "3.10.1",
+  "version": "3.10.2",
   "private": false,
   "type": "module",
   "main": "./cjs/index.js",