Skip to content

Commit d052e97

Browse files
committed
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.
1 parent d318ddd commit d052e97

1 file changed

Lines changed: 2 additions & 2 deletions

File tree

packages/angular/ssr/node/src/app-engine.ts

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -38,8 +38,8 @@ export class AngularNodeAppEngine {
3838
constructor(options?: AngularNodeAppEngineOptions) {
3939
const appEngineOptions: AngularAppEngineOptions = {
4040
...options,
41-
allowedHosts: getAllowedHostsFromEnv() ?? options?.allowedHosts,
42-
trustProxyHeaders: getTrustProxyHeadersFromEnv() ?? options?.trustProxyHeaders,
41+
allowedHosts: options?.allowedHosts ?? getAllowedHostsFromEnv(),
42+
trustProxyHeaders: options?.trustProxyHeaders ?? getTrustProxyHeadersFromEnv(),
4343
};
4444

4545
this.angularAppEngine = new AngularAppEngine(appEngineOptions);

0 commit comments

Comments
 (0)