-
Notifications
You must be signed in to change notification settings - Fork 1.3k
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Make proof param fetching resilient to single points of failure #12273
Comments
lotus gets proof from https://github.com/filecoin-project/go-paramfetch/blob/master/paramfetch.go |
doesn't the IPFS_GATEWAY env var already do that? |
https://github.com/ChainSafe/forest/blob/main/src/utils/proofs_api/paramfetch.rs#L37-L39C1 hardcodes the paths into if else if that fails it falls back to fetch from flip that around to fetch default and download from cf as a fallback or something like |
Yah I think the question is whether users would like lotus to provide at least 2 trusted options - which I thought could be nice when I opened issue |
yes, users would like that. i am not sure if the forest download is simple http or something r2 specific. we host them on http internally and move them pre lotus setup. lotus/go-paramfetch could integrate that and default to forest cloudflare like |
It's over HTTP, see here. So, in short, |
A couple of thoughts here:
I changed the title to "Make proof param fetching resilient to single points of failure" to account for the broader scope. I agree this is a "good first issue". |
Also, please note that we mirrored only the keys (based on our needs) - |
there is another source we are managing that is |
Checklist
Ideas
.Lotus component
What is the motivation behind this feature request? Is your feature request related to a problem? Please describe.
user reported proof param fetch failure from proof.filecoin.io due to a connection error (the host was down).
more details https://filecoinproject.slack.com/archives/CPFTWMY7N/p1721496569980859
Describe the solution you'd like
implement a fallback option to avoid central point of failure , i.e: fetch from the cloudflare forest team host
https://github.com/ChainSafe/forest/blob/main/src/utils/proofs_api/paramfetch.rs#L140-L145
Describe alternatives you've considered
No response
Additional context
No response
The text was updated successfully, but these errors were encountered: