Skip to content

fix(cli): warn when --api overrides configured API listen address - #13670

Merged
rjan90 merged 2 commits into
filecoin-project:masterfrom
DeepCogNeural:codex/warn-api-overrides-listenaddress
Jun 22, 2026
Merged

fix(cli): warn when --api overrides configured API listen address#13670
rjan90 merged 2 commits into
filecoin-project:masterfrom
DeepCogNeural:codex/warn-api-overrides-listenaddress

Conversation

@DeepCogNeural

Copy link
Copy Markdown
Contributor

Summary

  • warn when lotus daemon --api overrides the configured API.ListenAddress
  • include both the configured listen address and the endpoint selected from --api
  • add focused tests for warning and no-warning behavior

Fixes #12545

Testing

  • gofmt -l cli/lotus/daemon.go cli/lotus/daemon_test.go
  • git diff --check
  • CGO_LDFLAGS="-L/opt/homebrew/lib" go test ./cli/lotus -run TestAPIFlagOverrideWarning -count=1
  • CGO_LDFLAGS="-L/opt/homebrew/lib" go test ./cli/lotus -count=1

@github-project-automation github-project-automation Bot moved this to 📌 Triage in FilOz Jun 19, 2026

@github-actions github-actions Bot left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@github-project-automation github-project-automation Bot moved this from 📌 Triage to ⌨️ In Progress in FilOz Jun 19, 2026
@DeepCogNeural DeepCogNeural changed the title Warn when --api overrides configured API listen address fix(cli): warn when --api overrides configured API listen address Jun 19, 2026
@github-actions
github-actions Bot dismissed their stale review June 19, 2026 17:17

PR title now matches the required format.

@github-project-automation github-project-automation Bot moved this from ⌨️ In Progress to ✔️ Approved by reviewer in FilOz Jun 22, 2026
@rjan90
rjan90 merged commit 25a03f5 into filecoin-project:master Jun 22, 2026
179 of 180 checks passed
@github-project-automation github-project-automation Bot moved this from ✔️ Approved by reviewer to 🎉 Done in FilOz Jun 22, 2026
rjan90 pushed a commit that referenced this pull request Jun 30, 2026
…3670)

* Warn on API listen address override

* docs: add changelog for API override warning

(cherry picked from commit 25a03f5)
@rjan90 rjan90 mentioned this pull request Jun 30, 2026
43 tasks
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

Status: 🎉 Done

Development

Successfully merging this pull request may close these issues.

Lotus daemon --api setting overrides LOTUS_API_LISTENADDRESS without warning

3 participants