Skip to content

Fix: Remove hardcoded engine-auth secrets and secure operator deployment paths - #1171

Open
magqqgq wants to merge 4 commits into
base:mainfrom
magqqgq:magqqgq-patch-1
Open

Fix: Remove hardcoded engine-auth secrets and secure operator deployment paths#1171
magqqgq wants to merge 4 commits into
base:mainfrom
magqqgq:magqqgq-patch-1

Conversation

@magqqgq

Copy link
Copy Markdown

Description

This PR eliminates hardcoded secrets and hardens the configuration management practices within the node repository.

Vulnerabilities & Security Defects Remediated:

  • Secrets and Configuration Management (.env.mainnet, .env.sepolia, .gitignore, README.md): A live 64-character engine-auth credential was identified in the tracked deployment environments. This has been remediated by replacing the hardcoded secret with an explicit REPLACE_WITH_A_64_CHARACTER_HEX_SECRET placeholder. The .gitignore policy has been strengthened to ignore local configuration copies (.env, .env.*). The operator setup commands in README.md now correctly instruct users to define BASE_NODE_L2_ENGINE_AUTH_RAW only within untracked, local configuration files.

…ent paths
### Description
This PR eliminates hardcoded secrets and hardens the configuration management practices within the `node` repository.
**Vulnerabilities & Security Defects Remediated:**
* **Secrets and Configuration Management (`.env.mainnet`, `.env.sepolia`, `.gitignore`, `README.md`):** A live 64-character `engine-auth` credential was identified in the tracked deployment environments. This has been remediated by replacing the hardcoded secret with an explicit `REPLACE_WITH_A_64_CHARACTER_HEX_SECRET` placeholder. The `.gitignore` policy has been strengthened to ignore local configuration copies (`.env`, `.env.*`). The operator setup commands in `README.md` now correctly instruct users to define `BASE_NODE_L2_ENGINE_AUTH_RAW` only within untracked, local configuration files.
@cb-heimdall

Copy link
Copy Markdown
Collaborator

🟡 Heimdall Review Status

RequirementStatusMore Info
Reviews 🟡 0/1
Denominator calculation
Show calculation
1 if user is bot0
1 if user is external0
2 if repo is sensitive0
From .codeflow.yml1
Additional review requirements
Show calculation
Max0
0
From CODEOWNERS0
Global minimum0
Max 1
1
1 if commit is unverified0
Sum1

Sign up for freeto 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.

2 participants

@magqqgq@cb-heimdall