AgregoreWeb / agregore-browser

A minimal browser for the distributed web (Desktop version)
https://www.youtube.com/watch?v=TnYKvOQB0ts&list=PL7sG5SCUNyeYx8wnfMOUpsh7rM_g0w_cu&index=14
GNU Affero General Public License v3.0
715 stars 66 forks source link

Auto convert "ipfs://Qm..." into "ipfs://bafy or base32" #202

Open SLT-World opened 1 year ago

SLT-World commented 1 year ago

CID v0 doesn't work since Agregore Browser's engine, Chromium, as it turns out, converts URLs into lowercase. And IPFS does not work with lowercase CID v0. It might be a useful feature to detect CID v0 and auto convert it into CID v1.

RangerMauve commented 1 year ago

Do note that posting /ipfs/Qm... CIDs should resolve and convert automatically. It's just when you have ipfs://Qm URLs that it barfs.

RangerMauve commented 1 year ago

If anyone wants to take a shot at it. it should be a pretty straightforard change inside omni-box.js to add a case for strings that start with ipfs://Qm and to convert the CID