Skip to content

Replace startup-wide sed config rewriting with safer runtime config strategy#26

Open
KinshukSS2 wants to merge 1 commit intoistSOS:mainfrom
KinshukSS2:feature/fix-prod-runtime-config-rewrite
Open

Replace startup-wide sed config rewriting with safer runtime config strategy#26
KinshukSS2 wants to merge 1 commit intoistSOS:mainfrom
KinshukSS2:feature/fix-prod-runtime-config-rewrite

Conversation

@KinshukSS2
Copy link
Copy Markdown

@KinshukSS2 KinshukSS2 commented Apr 13, 2026

Proposed Changes

  • Remove full-tree find | sed -i mutation approach.
  • Introduce explicit runtime config mechanism (server-side env usage and/or generated config asset consumed by app).
  • Keep basePath/API behavior equivalent for deployment targets.
  • Add guardrails to avoid mutating unrelated files at startup.

details reference (fix #25)

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.

Production startup rewrites app files via broad find | sed

1 participant