+ ignore env vars with values not string

Author: linhntaim

README.md

@@ -554,7 +554,7 @@ a string contains `${value}` separated by commas;
 the `value` could be `null`,`boolean`, `number`, `"string"`, `[..array..]` or `{..object..}`;
 and all could be wrapped or not wrapped by `[` and `]` (*must*, when the string is empty).
 
-*Special case:* Values that have only a string enclosed by double quotes 
+*Special case:* Values that have only a string enclosed by double quotes
 also match the above format.
 
 *Spaces will be trimmed.*
@@ -1752,22 +1752,22 @@ See [this feature](#prevent-variables-from-conversion).
 
 Type: `object`. *Default:* `{}`.
 
-List of custom conversions for specific environment variables.
+Contains custom conversions for specific environment variables.
 
 See [this feature](#custom-conversion-for-a-specific-variable).
 
 ##### `methods`
 
 Type: `object`. *Default:* `{}`.
 
-List of custom conversion methods.
+Contains custom conversion methods.
 
 See [this feature](#custom-methods).
 
 ##### `methodAliases`
 
 Type: `object`. *Default:* `{}`.
 
-List of conversion method aliases.
+Contains conversion method aliases.
 
 See [this feature](#method-aliases).

dist/index.js

@@ -129,12 +129,15 @@ function parseSymbol(str) {
 
 /**
  *
- * @param {string} value
+ * @param {string|*} value
  * @param {object} valueTable
  * @param {boolean} fromDotEnv
  * @returns {null|undefined|boolean|number|bigint|string|symbol|array|object}
  */
 function restoreValue(value, valueTable, fromDotEnv) {
+  if (!(typeof value == 'string' || value instanceof String)) {
+    return value;
+  }
   if (fromDotEnv) {
     value = unescapeValue(value);
   }

src/index.js

@@ -129,12 +129,15 @@ function parseSymbol(str) {
 
 /**
  *
- * @param {string} value
+ * @param {string|*} value
  * @param {object} valueTable
  * @param {boolean} fromDotEnv
  * @returns {null|undefined|boolean|number|bigint|string|symbol|array|object}
  */
 function restoreValue(value, valueTable, fromDotEnv) {
+    if (!(typeof value == 'string' || value instanceof String)) {
+        return value
+    }
     if (fromDotEnv) {
         value = unescapeValue(value)
     }

tests/index.js

@@ -74,7 +74,11 @@ describe('dotenv-conversion', function () {
         it('set', function (done) {
             // input
             const inputConfig = {
-                parsed: {},
+                parsed: {
+                    TRUE: true,
+                    FALSE: 'false',
+                    NUMBER: new String('4.5e10'),
+                },
                 fromDotEnv: false,
                 ignoreProcessEnv: true,
                 prevents: ['BASIC'],
@@ -85,8 +89,8 @@ describe('dotenv-conversion', function () {
                 },
                 methods: {
                     // override existing built-in conversion method
-                    auto(value) {
-                        return 'auto'
+                    number(value) {
+                        return 'number'
                     },
                     // add new conversion method
                     basic(value) {
@@ -102,7 +106,11 @@ describe('dotenv-conversion', function () {
             }
 
             // output
-            const expectedParsed = inputConfig.parsed
+            const expectedParsed = {
+                TRUE: true,
+                FALSE: false,
+                NUMBER: 45000000000,
+            }
             const expectedFromDotEnv = inputConfig.fromDotEnv
             const expectedIgnoreProcessEnv = inputConfig.ignoreProcessEnv
             const expectedPrevents = inputConfig.prevents
@@ -119,7 +127,7 @@ describe('dotenv-conversion', function () {
 
                 'basic',
             ]
-            const expectedMethodAutoReturn = 'auto'
+            const expectedMethodNumberReturn = 'number'
             const expectedMethodBasicReturn = 'basic'
             const expectedMethodAliases = {
                 bool: 'boolean',
@@ -150,7 +158,7 @@ describe('dotenv-conversion', function () {
             dotenvConversionConfig.specs.should.deep.equal(expectedSpecs)
             Object.keys(dotenvConversionConfig.methods).should.deep.equal(expectedMethods)
             dotenvConversionConfig.methodAliases.should.deep.equal(expectedMethodAliases)
-            dotenvConversionConfig.methods.auto('value').should.equal(expectedMethodAutoReturn)
+            dotenvConversionConfig.methods.number('value').should.equal(expectedMethodNumberReturn)
             dotenvConversionConfig.methods.basic('value').should.equal(expectedMethodBasicReturn)
 
             done()