fix(config): fix number check for option value (#77)

* build(npm-lock): rename package-lock to npm-shrinkwrap * fix(config): fix number check for option value
svrxjs · Sep 2, 2019 · d82a13c · d82a13c
1 parent c012bbd
commit d82a13c
Show file tree

Hide file tree

Showing 2 changed files with 11 additions and 1 deletion.
diff --git a/packages/svrx/__tests__/spec/svrx.configure.js b/packages/svrx/__tests__/spec/svrx.configure.js
@@ -244,6 +244,16 @@ describe('CLI Config', () => {
       expect(plugin.get('foo')).to.eql('');
       expect(plugin.get('bar')).to.eql('foo');
     });
+
+    it('should parse web url with number correctly', () => {
+      const server = createServer({}, {
+        plugin: [
+          'test?host=https://test.163.com',
+        ],
+      });
+      const plugin = server.config.getPlugin('test');
+      expect(plugin.get('host')).to.eql('https://test.163.com');
+    });
   });
 
   describe('add plugin with shortcut', () => {

diff --git a/packages/svrx/lib/configure/index.js b/packages/svrx/lib/configure/index.js
@@ -204,7 +204,7 @@ class Configure {
       };
       Object.keys(ops).forEach((key) => {
         const value = ops[key];
-        if (/\d+/.test(value)) {
+        if (/^\d+$/.test(value)) {
           newOps[key] = parseInt(value, 10);
         } else if (value in keyMap) {
           newOps[key] = keyMap[value];