Skip to content

repo: use config api to get node root path#10934

Merged
gammazero merged 2 commits intomasterfrom
config-api-for-path
Sep 2, 2025
Merged

repo: use config api to get node root path#10934
gammazero merged 2 commits intomasterfrom
config-api-for-path

Conversation

@gammazero
Copy link
Contributor

@gammazero gammazero commented Aug 22, 2025

Unifies how kubo gets the value of IPFS_PATH by using config.PathRoot

Replaces #8964

Closes #8848

Contains additional fixes for test code, such as os.Setenv => t.Setenv

@gammazero gammazero added the skip/changelog This change does NOT require a changelog entry label Aug 22, 2025
@gammazero gammazero requested a review from a team as a code owner August 22, 2025 23:28
@gammazero gammazero added the need/triage Needs initial labeling and prioritization label Aug 26, 2025
@lidel lidel mentioned this pull request Aug 26, 2025
49 tasks
@gammazero gammazero merged commit 17b0085 into master Sep 2, 2025
16 checks passed
@gammazero gammazero deleted the config-api-for-path branch September 2, 2025 20:13
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

need/triage Needs initial labeling and prioritization skip/changelog This change does NOT require a changelog entry

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Replace os.Getenv("HOME")+IPFS_PATH with safer config.PathRoot

2 participants