Skip to content

Fix missing config #192

Merged
gjeanmart merged 2 commits intomainfrom
fix/missing-config
Mar 6, 2026
Merged

Fix missing config #192
gjeanmart merged 2 commits intomainfrom
fix/missing-config

Conversation

@gjeanmart
Copy link
Collaborator

Fix missing config

Summary

This PR fixes several missing or outdated configuration entries in the environment files and docker-compose.yml to improve the out-of-the-box self-hosted experience.

Changes

container_env_files/cgw.env

  • Add ALLOW_CORS=true to enable CORS support
  • Add missing RELAY_PROVIDER_API_KEY_UNICHAIN relay key placeholder
  • Add dummy CSV_AWS_* credentials required for CSV export (LocalStack-compatible)
  • Add BLOCKLIST_ENCRYPTED_DATA, BLOCKLIST_SECRET_KEY, and BLOCKLIST_SECRET_SALT
    placeholders for the address blocklist feature

container_env_files/ui.env

  • Add NEXT_PUBLIC_DEFAULT_MAINNET_CHAIN_ID for chain-specific deployments
  • Replace deprecated NEXT_PUBLIC_WC_BRIDGE with NEXT_PUBLIC_WC_PROJECT_ID
    (WalletConnect v2)

docker-compose.yml

  • Fix circular dependency: remove depends_on: nginx from the ui service and instead add ui to nginx's healthcheck depends_on, so nginx waits for the UI to be ready before starting

docs/chain_info.md

  • Add a Features section listing the required chain features to enable in the CFG admin panel (MY_ACCOUNTS, SPENDING_LIMIT, SAFE_APPS, ERC1155, ERC721, DOMAIN_LOOKUP, CONTRACT_INTERACTION)

- Add missing CGW env vars: ALLOW_CORS, RELAY_PROVIDER_API_KEY_UNICHAIN, blocklist config, and dummy AWS credentials for CSV export
- Add NEXT_PUBLIC_DEFAULT_MAINNET_CHAIN_ID to ui.env for dedicated chain deployments
- Rename deprecated NEXT_PUBLIC_WC_BRIDGE to NEXT_PUBLIC_WC_PROJECT_ID
- Fix circular dependency: move ui from nginx depends_on to nginx healthcheck
- Document required chain features in docs/chain_info.md
@gjeanmart gjeanmart requested a review from Uxio0 March 2, 2026 11:17
@gjeanmart gjeanmart self-assigned this Mar 2, 2026
@github-actions
Copy link

github-actions bot commented Mar 2, 2026

CLA Assistant Lite bot All contributors have signed the CLA ✍️ ✅

@gjeanmart
Copy link
Collaborator Author

I have read the CLA Document and I hereby sign the CLA

github-actions bot added a commit that referenced this pull request Mar 2, 2026
@gjeanmart gjeanmart merged commit 42050ca into main Mar 6, 2026
2 of 3 checks passed
@gjeanmart gjeanmart deleted the fix/missing-config branch March 6, 2026 12:16
@github-actions github-actions bot locked and limited conversation to collaborators Mar 6, 2026
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants