fix(@angular/build): resolve dev-server sslKey and sslCert relative to the workspace root (#34399)

The Vite-based dev server read `sslKey` and `sslCert` relative to the current working directory,
so serving failed with `ENOENT` when `ng serve` ran from a subdirectory of the workspace and the
paths in `angular.json` were relative. Resolve both paths against the workspace root, as the
webpack dev server already does.

The new dev-server `ssl` spec covers relative paths. It fails with `ENOENT` without this change.

PR Close #34399

Author: MeAkib

packages/angular/build/src/builders/dev-server/options.ts

@@ -142,8 +142,8 @@ export async function normalizeOptions(
     proxyConfig,
     servePath,
     ssl,
-    sslCert,
-    sslKey,
+    sslCert: sslCert && path.resolve(workspaceRoot, sslCert),
+    sslKey: sslKey && path.resolve(workspaceRoot, sslKey),
     // Prebundling defaults to true but requires caching to function
     prebundle: cacheOptions.enabled && !optimization.scripts && prebundle,
     inspect,

packages/angular/build/src/builders/dev-server/tests/options/ssl_spec.ts

@@ -0,0 +1,84 @@
+/**
+ * @license
+ * Copyright Google LLC All Rights Reserved.
+ *
+ * Use of this source code is governed by an MIT-style license that can be
+ * found in the LICENSE file at https://angular.dev/license
+ */
+
+import assert from 'node:assert';
+import { mkdtemp, writeFile } from 'node:fs/promises';
+import { join } from 'node:path';
+import { executeDevServer } from '../../index';
+import { describeServeBuilder } from '../jasmine-helpers';
+import { BASE_OPTIONS, DEV_SERVER_BUILDER_INFO } from '../setup';
+
+const SSL_KEY = `-----BEGIN PRIVATE KEY-----
+MIGHAgEAMBMGByqGSM49AgEGCCqGSM49AwEHBG0wawIBAQQgEpythQjrJo9bm8GM
+LFacGutCqc/XxDZ3dDkSMYZYZ2ehRANCAARvwtZ3OwP1uLhrcLp7nemvaWqkA+FN
+NDjOA3OZKhyVHYog1jNTawJJjwk66upbXGFL8622yU1INWbZFxvrSANl
+-----END PRIVATE KEY-----
+`;
+
+const SSL_CERT = `-----BEGIN CERTIFICATE-----
+MIIBfzCCASWgAwIBAgIUUoRhwzxAE3y5nrOW8de+cJEIVvQwCgYIKoZIzj0EAwIw
+FDESMBAGA1UEAwwJbG9jYWxob3N0MCAXDTI2MTAxOTA4NTUyNVoYDzIxMjYwOTI1
+MDg1NTI1WjAUMRIwEAYDVQQDDAlsb2NhbGhvc3QwWTATBgcqhkjOPQIBBggqhkjO
+PQMBBwNCAARvwtZ3OwP1uLhrcLp7nemvaWqkA+FNNDjOA3OZKhyVHYog1jNTawJJ
+jwk66upbXGFL8622yU1INWbZFxvrSANlo1MwUTAdBgNVHQ4EFgQUWyqB1QeFcs6L
+LL7Utsybm6+XFvIwHwYDVR0jBBgwFoAUWyqB1QeFcs6LLL7Utsybm6+XFvIwDwYD
+VR0TAQH/BAUwAwEB/zAKBggqhkjOPQQDAgNIADBFAiBazry25W+Qewb/WzadhmdY
+YDGdshNZCk0g0DI3ctNjlgIhAKm2X8pmqdkM38Yh8QDN0cHBsAcDfRYffDnjJ3tP
+dpnU
+-----END CERTIFICATE-----
+`;
+
+describeServeBuilder(executeDevServer, DEV_SERVER_BUILDER_INFO, (harness, setupTarget) => {
+  describe('option: "ssl"', () => {
+    beforeEach(async () => {
+      setupTarget(harness);
+
+      // Application code is not needed for these tests
+      await harness.writeFile('src/main.ts', '');
+    });
+
+    it('resolves "sslKey" and "sslCert" relative to the workspace root', async () => {
+      await harness.writeFile('ssl/server.key', SSL_KEY);
+      await harness.writeFile('ssl/server.crt', SSL_CERT);
+
+      harness.useTarget('serve', {
+        ...BASE_OPTIONS,
+        ssl: true,
+        sslKey: 'ssl/server.key',
+        sslCert: 'ssl/server.crt',
+      });
+
+      const { result } = await harness.executeOnce();
+
+      expect(result?.success).toBeTrue();
+      expect(result?.baseUrl).toMatch(/^https:\/\/[^/]+:\d+\//);
+    });
+
+    it('supports absolute "sslKey" and "sslCert" paths outside the workspace', async () => {
+      const baseTmpDir = process.env['TEST_TMPDIR'];
+      assert(baseTmpDir, 'TEST_TMPDIR is not set');
+      const certsDir = await mkdtemp(join(baseTmpDir, 'angular-cli-ssl-'));
+      const sslKey = join(certsDir, 'server.key');
+      const sslCert = join(certsDir, 'server.crt');
+      await writeFile(sslKey, SSL_KEY);
+      await writeFile(sslCert, SSL_CERT);
+
+      harness.useTarget('serve', {
+        ...BASE_OPTIONS,
+        ssl: true,
+        sslKey,
+        sslCert,
+      });
+
+      const { result } = await harness.executeOnce();
+
+      expect(result?.success).toBeTrue();
+      expect(result?.baseUrl).toMatch(/^https:\/\/[^/]+:\d+\//);
+    });
+  });
+});