token: Fix CIDR list processing to handle comma separated items correctly

This got broken by having the config-type set to Array.  When it was a
String, it worked correctly.

1 files changed, 19 insertions, 0 deletions

diff --git a/test/tap/unit-token-validate-cidr.js b/test/tap/unit-token-validate-cidr.js
new file mode 100644
index 000000000..db963c31f
--- /dev/null
+++ b/test/tap/unit-token-validate-cidr.js
@@ -0,0 +1,19 @@
+'use strict'
+const test = require('tap').test
+const requireInject = require('require-inject')
+const validateCIDRList = requireInject('../../lib/token.js', {'../../lib/npm.js': {}})._validateCIDRList
+
+test('validateCIDRList', (t) => {
+  t.plan(10)
+  const single = ['127.0.0.0/24']
+  const double = ['127.0.0.0/24', '192.168.0.0/16']
+  const ipv6 = '2620:0:2d0:200::7/32'
+  const ipv6Mixed = ['127.0.0/24', '2620:0:2d0:200::7/32', '192.168.0.0/16']
+  t.doesNotThrow(() => t.isDeeply(validateCIDRList(single.join(',')), single), 'single string ipv4')
+  t.doesNotThrow(() => t.isDeeply(validateCIDRList(single), single), 'single array ipv4')
+  t.doesNotThrow(() => t.isDeeply(validateCIDRList(double.join(',')), double), 'double string ipv4')
+  t.doesNotThrow(() => t.isDeeply(validateCIDRList(double), double), 'double array ipv4')
+  t.throws(() => validateCIDRList(ipv6))
+  t.throws(() => validateCIDRList(ipv6Mixed))
+  t.done()
+})