-
Notifications
You must be signed in to change notification settings - Fork 106
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
Update Helia to fallback to trustless HTTP gateways #274
Comments
#276 is still in progress, but I could use some input from @achingbrain that the direction i'm heading in works before I flesh everything out |
The work here is done and merged into the |
@SgtPooki : what are the conditions for closing this?
Thanks |
Yep. I discussed this with Alex and we talked about how we would let this sit in
Docs are not published yet but should be in https://helia.io/interfaces/helia.HeliaInit.html & https://helia.io/modules/_helia_interface.blocks.html when they are. You can see the docs that will be mentioned in the source here:
It could be worth adding some info to the wiki or README including the diagrams at #272 once we decide we will release these changes. |
I can actually confirm the trustless gateway works. The only issue are some of the gateways don't properly allow CORS which causes issues in browser when fetching CAR data. |
@pcfreak30 thanks for the update! Which gateways were causing issues? |
I would honestly test them all. I am using a rather complex system that is powered by webworkers inside iframes. The CORS should be such it allows everything. |
Yea, will do. FYI, you can limit the gateway urls you enable by setting
|
FYI that I threw a quick codepen together that shows this working fine for all default gateways: https://codepen.io/SgtPooki/pen/JjxYpZX |
I will update if I can replicate in my setup, thanks. |
@SgtPooki i was able to get a screenshot from a tester. |
@SgtPooki : what is needed for closing out this issue? |
I believe this is done except for #374. The spec work for other gateways is at ipfs/specs#423. closing this |
discussed in https://pl-strflt.notion.site/Helia-reliable-retrieval-technical-design-golden-path-255-15c4d3c25a404a85b6db8bf3f8d1f310?pvs=4
Should do #272 first
Tasks
BlockBroker
API changes and retrievability improvements #287The text was updated successfully, but these errors were encountered: