-
-
Notifications
You must be signed in to change notification settings - Fork 3.1k
Closed
Labels
P2Medium: Good to have, but can wait until someone steps upMedium: Good to have, but can wait until someone steps upexp/intermediatePrior experience is likely helpfulPrior experience is likely helpfulhelp wantedSeeking public contribution on this issueSeeking public contribution on this issuekind/bugA bug in existing code (including security flaws)A bug in existing code (including security flaws)topic/daemon + init
Description
We should:
- Atomically create the
IPFS_HOME/apifile to ensure we never leave an empty/corrupt one behind. The relevant code is in the SetAPIAddr function in repo/fsrepo/fsrepo.go. We should create a temporary file, write to the temporary file, then atomically rename to "api". - If we read the "api" file and it's empty, we should ignore it.
Reported on the forums: https://discuss.ipfs.io/t/error-multiaddr-must-start-with-a/7824
Reactions are currently unavailable
Metadata
Metadata
Assignees
Labels
P2Medium: Good to have, but can wait until someone steps upMedium: Good to have, but can wait until someone steps upexp/intermediatePrior experience is likely helpfulPrior experience is likely helpfulhelp wantedSeeking public contribution on this issueSeeking public contribution on this issuekind/bugA bug in existing code (including security flaws)A bug in existing code (including security flaws)topic/daemon + init