Skip to content

fix(build): Set EJS download to --noproxy to fix auth-server startup problem#20365

Open
LZoog wants to merge 1 commit intomainfrom
fix-startup-issue
Open

fix(build): Set EJS download to --noproxy to fix auth-server startup problem#20365
LZoog wants to merge 1 commit intomainfrom
fix-startup-issue

Conversation

@LZoog
Copy link
Copy Markdown
Contributor

@LZoog LZoog commented Apr 13, 2026

Not sure if anyone else is having this problem, but my stack would not start up today and I traced it down to auth-server not starting due to this script outputting this to vendor/ejs.js:

<html><body><h1>504 Gateway Time-out</h1>
The server didn't respond in time.
</body></html>

Manually curling / running the command worked, but running nx build on auth-server or starting up the stack, would fail on this vendor/ejs.js having "no default export," because the file was the above.

@LZoog LZoog requested a review from a team as a code owner April 13, 2026 17:45
# Download it from github
curl -L https://github.com/mde/ejs/releases/download/v$ejs_version/ejs.min.js -o ./vendor/ejs.js
# Use --noproxy to avoid yarn/npm proxy env vars causing 504 gateway timeouts
curl -L --noproxy '*' https://github.com/mde/ejs/releases/download/v$ejs_version/ejs.min.js -o ./vendor/ejs.js
Copy link
Copy Markdown
Contributor

@MagentaManifold MagentaManifold Apr 13, 2026

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Suggested change
curl -L --noproxy '*' https://github.com/mde/ejs/releases/download/v$ejs_version/ejs.min.js -o ./vendor/ejs.js
curl -fL --noproxy '*' https://github.com/mde/ejs/releases/download/v$ejs_version/ejs.min.js -o ./vendor/ejs.js

Nit: -f makes curl fail on HTTP errors instead of saving garbage

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Nice, thanks!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants