From 42adc2dc6d73d4d64ace7b41c5acbb43f98aaab3 Mon Sep 17 00:00:00 2001 From: Alan Agius <17563226+alan-agius4@users.noreply.github.com> Date: Thu, 11 Jun 2026 09:04:20 +0000 Subject: [PATCH] fix(@angular/ssr): prioritize options over environment variables in AngularNodeAppEngine Prioritize constructor options over environment variables when initializing the AngularNodeAppEngine. Previously, environment variables took priority and blindly overrode the constructor options if they were defined in the environment. Now, explicit constructor options act as the override, while the environment variables serve as a fallback. --- packages/angular/ssr/node/src/app-engine.ts | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/packages/angular/ssr/node/src/app-engine.ts b/packages/angular/ssr/node/src/app-engine.ts index 55f4222d4a88..4bb6d28bb045 100644 --- a/packages/angular/ssr/node/src/app-engine.ts +++ b/packages/angular/ssr/node/src/app-engine.ts @@ -38,8 +38,8 @@ export class AngularNodeAppEngine { constructor(options?: AngularNodeAppEngineOptions) { const appEngineOptions: AngularAppEngineOptions = { ...options, - allowedHosts: getAllowedHostsFromEnv() ?? options?.allowedHosts, - trustProxyHeaders: getTrustProxyHeadersFromEnv() ?? options?.trustProxyHeaders, + allowedHosts: options?.allowedHosts ?? getAllowedHostsFromEnv(), + trustProxyHeaders: options?.trustProxyHeaders ?? getTrustProxyHeadersFromEnv(), }; this.angularAppEngine = new AngularAppEngine(appEngineOptions);