Skip to content

improvement(dockerfile): increase Node.js memory limit from 1024MB to 2048MB#5519

Merged
victorvhs017 merged 1 commit intomainfrom
improvement/update-standalone-image-node-memory-2gb
Feb 20, 2026
Merged

improvement(dockerfile): increase Node.js memory limit from 1024MB to 2048MB#5519
victorvhs017 merged 1 commit intomainfrom
improvement/update-standalone-image-node-memory-2gb

Conversation

@victorvhs017
Copy link
Contributor

Context

Set the default Node memory limit to 2GB.

Type

  • Fix
  • Feature
  • Improvement
  • Breaking
  • Docs
  • Chore

Checklist

  • Title follows the conventional commit format: type(scope): short description (scope is optional, e.g., fix: prevent crash on sync or fix(api): handle null response).
  • Tested locally
  • Updated docs (if needed)
  • Read the contributing guide

@maidul98
Copy link
Collaborator

Snyk checks have passed. No issues have been found so far.

Status Scanner Critical High Medium Low Total (0)
Open Source Security 0 0 0 0 0 issues

💻 Catch issues earlier using the plugins for VS Code, JetBrains IDEs, Visual Studio, and Eclipse.

@victorvhs017 victorvhs017 changed the title improvement(Dockerfile): increase Node.js memory limit from 1024MB to 2048MB improvement(dockerfile): increase Node.js memory limit from 1024MB to 2048MB Feb 19, 2026
@greptile-apps
Copy link
Contributor

greptile-apps bot commented Feb 19, 2026

Greptile Summary

Increases the Node.js memory limit for the production runtime from 1024MB to 2048MB in the standalone Docker image. This change aligns better with the documented hardware requirements (4-8 GB per container) and provides more headroom for memory-intensive operations.

Key observations:

  • Change only affects the production runtime environment (NODE_OPTIONS on line 224)
  • Build stages already use 8192MB (--max-old-space-size=8192 on lines 41 and 87), so build performance is unaffected
  • Documented requirements recommend 4-8 GB memory per container, so 2GB Node heap is reasonable
  • No breaking changes or API modifications

Confidence Score: 5/5

  • This PR is safe to merge with no risk
  • Single line configuration change that increases memory allocation - no code logic changes, no security implications, and aligns with documented hardware requirements
  • No files require special attention

Important Files Changed

Filename Overview
Dockerfile.standalone-infisical Increases Node.js runtime memory limit from 1GB to 2GB for production environment - straightforward configuration change with no logic issues

Last reviewed commit: 7e349a2

@victorvhs017 victorvhs017 merged commit 974f832 into main Feb 20, 2026
7 of 8 checks passed
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