Skip to content
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

Use ipfs:// instead of fs://ipfs #20

Closed
lidel opened this issue May 16, 2019 · 2 comments
Closed

Use ipfs:// instead of fs://ipfs #20

lidel opened this issue May 16, 2019 · 2 comments

Comments

@lidel
Copy link

lidel commented May 16, 2019

Hi, this is a drive-by comment from IPFS 🚗 💨

The fs:/ was droped as an idea some years ago.
Since then we switched to ipfs://, ipns:// and dweb:/
See ipfs/in-web-browsers/ADDRESSING.md for up-to-date recommendations.

Would be fantastic to add archon support for ipfs:// and deprecate/remove fs:/ in code and docs.

@satello
Copy link
Contributor

satello commented May 17, 2019

Will do. Thanks for the clarifications in ethereum/EIPs#1497 (comment). For the purposes of this library we will only be dealing with immutable files (at the moment). The preferred URI scheme I have been using is /ipfs/{cid}. I had support for fs:/ and ipfs:/ to try to cover all standard file formats, I will update to ipfs:// and dweb:/ instead

@satello satello self-assigned this May 17, 2019
@lidel lidel changed the title Use dweb:/ instead of fs:/ Use ipfs:// instead of fs://ipfs Jul 20, 2023
@jaybuidl
Copy link
Member

Closing this. The current scheme is consistent with Protocol Lab's proposed Multiaddress scheme which appears superior to RFC3986 URIs.

Context: kleros/builder-baseline#16

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

3 participants