Skip to content

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

Open
mertcano wants to merge 1 commit into
giwa-io:mainfrom
mertcano:mertcano-patch-3
Open

Fix: Remove hardcoded engine-auth secrets and secure operator deployment paths#45
mertcano wants to merge 1 commit into
giwa-io:mainfrom
mertcano:mertcano-patch-3

Conversation

@mertcano

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.
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.

1 participant