feat: support subdomain matching via leading dot in no_proxy rule

Support subdomain matching via leading dot in no_proxy rule

src/utilities/isUrlMatchingNoProxy.js

@@ -14,7 +14,9 @@ export default (subjectUrl: string, noProxy: string) => {
   const rules = noProxy.split(/[,\s]/);
 
   for (const rule of rules) {
-    const ruleMatch = rule.match(/^(?<hostname>.+?)(?::(?<port>\d+))?$/);
+    const ruleMatch = rule
+      .replace(/^(?<leadingDot>\.)/, '*')
+      .match(/^(?<hostname>.+?)(?::(?<port>\d+))?$/);
 
     if (!ruleMatch || !ruleMatch.groups) {
       throw new UnexpectedStateError('Invalid NO_PROXY pattern.');

test/global-agent/utilities/isUrlMatchingNoProxy.js

@@ -11,10 +11,18 @@ test('returns `true` if hosts match (IP)', (t) => {
   t.assert(isUrlMatchingNoProxy('http://127.0.0.1/', '127.0.0.1'));
 });
 
-test('returns `true` if hosts match (using wildcard)', (t) => {
+test('returns `true` if hosts match (using asterisk wildcard)', (t) => {
   t.assert(isUrlMatchingNoProxy('http://bar.foo.com/', '*.foo.com'));
 });
 
+test('returns `true` if domain matches (using dot wildcard)', (t) => {
+  t.assert(isUrlMatchingNoProxy('http://foo.com/', '.foo.com'));
+});
+
+test('returns `true` if subdomain matches (using dot wildcard)', (t) => {
+  t.assert(isUrlMatchingNoProxy('http://bar.foo.com/', '.foo.com'));
+});
+
 test('returns `true` if hosts match (*) and ports match', (t) => {
   t.assert(isUrlMatchingNoProxy('http://foo.com:8080/', '*:8080'));
 });
@@ -42,3 +50,7 @@ test('returns `false` if hosts match and ports do not match (port not present su
 test('returns `true` if hosts match and ports do not match (port not present NO_PROXY)', (t) => {
   t.assert(isUrlMatchingNoProxy('http://foo.com:8000/', 'foo.com'));
 });
+
+test('returns `true` if hosts match in one of multiple rules', (t) => {
+  t.assert(isUrlMatchingNoProxy('http://foo.com/', 'bar.org,foo.com,baz.io'));
+});