.github/workflows/ci.yaml

@@ -19,13 +19,7 @@ jobs:
       - uses: actions/setup-node@v2
         with:
           node-version: ${{ matrix.node }}
-      - name: Set up npm cache
-        uses: actions/cache@v2
-        with:
-          path: ~/.npm
-          key: ${{ runner.os }}-node-v${{ matrix.node }}-${{ hashFiles('package.json') }}-${{ hashFiles('package-lock.json') }}
-          restore-keys: |
-            ${{ runner.os }}-node-v${{ matrix.node }}-${{ hashFiles('package.json') }}-${{ hashFiles('package-lock.json') }}
+          cache: npm
       - run: npm ci
       - run: npm test
       - run: npm run codecov
@@ -37,17 +31,7 @@ jobs:
       - uses: actions/setup-node@v2
         with:
           node-version: ${{ env.NODE }}
-      - name: Get npm cache directory
-        id: npm-cache
-        run: |
-          echo "::set-output name=dir::$(npm config get cache)"
-      - name: Set up npm cache
-        uses: actions/cache@v2
-        with:
-          path: ${{ steps.npm-cache.outputs.dir }}
-          key: ${{ runner.os }}-node-v${{ env.NODE }}-${{ hashFiles('package.json') }}-${{ hashFiles('package-lock.json') }}
-          restore-keys: |
-            ${{ runner.os }}-node-v${{ env.NODE }}-${{ hashFiles('package.json') }}-${{ hashFiles('package-lock.json') }}
+          cache: npm
       - run: npm ci
       - run: npm test
   lint:
@@ -57,13 +41,7 @@ jobs:
       - uses: actions/setup-node@v2
         with:
           node-version: ${{ env.NODE }}
-      - name: Set up npm cache
-        uses: actions/cache@v2
-        with:
-          path: ~/.npm
-          key: ${{ runner.os }}-node-v${{ env.NODE }}-${{ hashFiles('package.json') }}-${{ hashFiles('package-lock.json') }}
-          restore-keys: |
-            ${{ runner.os }}-node-v${{ env.NODE }}-${{ hashFiles('package.json') }}-${{ hashFiles('package-lock.json') }}
+          cache: npm
       - run: npm ci
       - run: npm run lint
   docs:
@@ -73,13 +51,7 @@ jobs:
       - uses: actions/setup-node@v2
         with:
           node-version: ${{ env.NODE }}
-      - name: Set up npm cache
-        uses: actions/cache@v2
-        with:
-          path: ~/.npm
-          key: ${{ runner.os }}-node-v${{ env.NODE }}-${{ hashFiles('package.json') }}-${{ hashFiles('package-lock.json') }}
-          restore-keys: |
-            ${{ runner.os }}-node-v${{ env.NODE }}-${{ hashFiles('package.json') }}-${{ hashFiles('package-lock.json') }}
+          cache: npm
       - run: npm ci
       - run: npm run docs-test
   release:
@@ -91,13 +63,7 @@ jobs:
       - uses: actions/setup-node@v2
         with:
           node-version: ${{ env.NODE }}
-      - name: Set up npm cache
-        uses: actions/cache@v2
-        with:
-          path: ~/.npm
-          key: ${{ runner.os }}-node-v${{ env.NODE }}-${{ hashFiles('package.json') }}-${{ hashFiles('package-lock.json') }}
-          restore-keys: |
-            ${{ runner.os }}-node-v${{ env.NODE }}-${{ hashFiles('package.json') }}-${{ hashFiles('package-lock.json') }}
+          cache: npm
       - run: npm ci
       - run: npm run compile
       - run: npm run build-binaries

.github/workflows/codeql.yml

@@ -9,26 +9,27 @@ on:
     # The branches below must be a subset of the branches above
     branches:
       - main
+      - "!renovate/**"
   schedule:
     - cron: "0 0 * * 0"
 
 jobs:
   analyze:
     name: Analyze
     runs-on: ubuntu-latest
+    permissions:
+      actions: read
+      contents: read
+      security-events: write
 
     steps:
       - name: Checkout repository
         uses: actions/checkout@v2
 
-      # Initializes the CodeQL tools for scanning.
       - name: Initialize CodeQL
         uses: github/codeql-action/init@v1
         with:
           languages: "javascript"
 
-      - name: Autobuild
-        uses: github/codeql-action/autobuild@v1
-
       - name: Perform CodeQL Analysis
         uses: github/codeql-action/analyze@v1

renovate.json

@@ -1,6 +1,7 @@
 {
   "extends": [
-    "config:base"
+    "config:base",
+    ":disableDependencyDashboard"
   ],
   "pinVersions": false,
   "rebaseStalePrs": true

src/index.ts

@@ -1,5 +1,6 @@
 import {EventEmitter} from 'events';
 import {URL} from 'url';
+import {AddressInfo} from 'net';
 import * as http from 'http';
 import * as path from 'path';
 import {Readable} from 'stream';
@@ -81,13 +82,17 @@ export class LinkChecker extends EventEmitter {
     let server: http.Server | undefined;
     const hasHttpPaths = options.path.find(x => x.startsWith('http'));
     if (!hasHttpPaths) {
-      const port = options.port || 5000 + Math.round(Math.random() * 1000);
+      let port = options.port;
       server = await startWebServer({
         root: options.serverRoot!,
         port,
         markdown: options.markdown,
         directoryListing: options.directoryListing,
       });
+      if (port === undefined) {
+        const addr = server.address() as AddressInfo;
+        port = addr.port;
+      }
       for (let i = 0; i < options.path.length; i++) {
         if (options.path[i].startsWith('/')) {
           options.path[i] = options.path[i].slice(1);

src/server.ts

@@ -1,3 +1,4 @@
+import {AddressInfo} from 'net';
 import * as http from 'http';
 import * as path from 'path';
 import * as fs from 'fs';
@@ -16,7 +17,7 @@ export interface WebServerOptions {
   // The local path that should be mounted as a static web server
   root: string;
   // The port on which to start the local web server
-  port: number;
+  port?: number;
   // If markdown should be automatically compiled and served
   markdown?: boolean;
   // Should directories automatically serve an inde page
@@ -33,8 +34,12 @@ export async function startWebServer(options: WebServerOptions) {
   return new Promise<http.Server>((resolve, reject) => {
     const server = http
       .createServer((req, res) => handleRequest(req, res, root, options))
-      .listen(options.port, () => resolve(server))
+      .listen(options.port || 0, () => resolve(server))
       .on('error', reject);
+    if (!options.port) {
+      const addr = server.address() as AddressInfo;
+      options.port = addr.port;
+    }
     enableDestroy(server);
   });
 }

test/test.cli.ts

@@ -10,7 +10,7 @@ import {LinkResult, LinkState} from '../src/index';
 // eslint-disable-next-line prefer-arrow-callback
 describe('cli', function () {
   let server: http.Server;
-  this.timeout(10_000);
+  this.timeout(20_000);
 
   const pkg = require('../../package.json');
   const linkinator = pkg.bin.linkinator;

test/test.server.ts

@@ -1,22 +1,23 @@
 import * as assert from 'assert';
 import {describe, it, before, after} from 'mocha';
 import {startWebServer} from '../src/server';
+import {AddressInfo} from 'net';
 import {Server} from 'http';
 import {request} from 'gaxios';
 import * as fs from 'fs';
 
 describe('server', () => {
   let server: Server;
-  const port = 5000 + Math.round(Math.random() * 1000);
-  const rootUrl = `http://localhost:${port}`;
+  let rootUrl: string;
   const contents = fs.readFileSync('test/fixtures/server/index.html', 'utf-8');
   before(async () => {
     server = await startWebServer({
-      port,
       directoryListing: true,
       markdown: true,
       root: 'test/fixtures/server',
     });
+    const addr = server.address() as AddressInfo;
+    rootUrl = `http://localhost:${addr.port}`;
   });
   after(() => server.destroy());