Skip to content

Commit c8088a5

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. (cherry picked from commit 5875b60)
1 parent 285a34e commit c8088a5

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)