fix: use only ipni-enabled providers #109
Merged
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Summary
Update
filecoin-pinto use the newwithIpniprovider filtering option from synapse-sdk. Here's what was changed:Files Modified:
src/core/synapse/index.ts(2 changes):DEFAULT_STORAGE_CONTEXT_CONFIGto includewithIpni: trueon line 32initializeSynapsefunction to passwithIpni: trueinSynapseOptionson line 227src/payments/interactive.ts(1 change):Synapse.create()call to includewithIpni: trueon line 95Key Points:
✅ All Synapse instances now pass
withIpni: trueto ensure only IPNI-enabled providers are selected✅ All storage contexts created through our wrapper functions will automatically have
withIpni: truein their default configuration✅ Upload-action doesn't need changes since it uses our
createStorageContextandinitializeSynapsefunctions which now have the defaults setConsistency with SDK Updates:
The changes align with the synapse-sdk PR #309 which:
withIpnifiltering to provider selectionforceCreateDataSetis trueprovider?.products.PDP?.data.ipniIpfs === falseFixes #104