Skip to content
This repository has been archived by the owner on Mar 22, 2022. It is now read-only.

feat: improved CID column (filename and dnslink support) #43

Merged
merged 2 commits into from
Sep 28, 2020

Conversation

lidel
Copy link
Member

@lidel lidel commented Sep 26, 2020

This PR closes #42 by taking advantage of DNSLink flag added in ipfs/kubo#7699

Changes:

  • Links to direct CIDs now have ?filename= param to preserve name and content type
  • When page is loaded on DNSLink website then there is no public gateway endpoint mounted.
    • We still want to keep CID column, so as a replacement, clicking on a CID opens http://cid.ipfs.io#<cid>
  • figured out why CI adds -e to <style> block, npm run build should now produce same output on all unix platforms
  • did end-to-end test with fix(dnslink-gw): breadcrumbs and CID column when dir listing kubo#7699 and things work as expected

- Links to direct CIDs now have ?filename= param to preserve name and content type
- When page is loaded on DNSLink website (based on Host header) then
  there is no public gateway endpoint.
  As a replacement, clicking on a CID opens it at cid.ipfs.io

Context: #42

Closes #42
for some reason `-e` gets interpreted literally at CI.
This removes its use making the output look the same everywhere.
CI will now show diff if uncommited changes to make it easier to see what
caused CI to fail.
lidel added a commit to ipfs/kubo that referenced this pull request Sep 28, 2020
This switches go-ipfs to dir-index-html from
ipfs/dir-index-html#43
@lidel
Copy link
Member Author

lidel commented Sep 28, 2020

@jessicaschilling I don't expect more changes here, so this is ready to be reviewed (and if looks good, to be released).
Then we can switch ipfs/kubo#7699 to release version :)

@lidel lidel marked this pull request as ready for review September 28, 2020 16:45
@jessicaschilling jessicaschilling merged commit 92efd3d into master Sep 28, 2020
@jessicaschilling jessicaschilling deleted the fix/links-when-dnslink branch September 28, 2020 19:47
lidel added a commit to ipfs/kubo that referenced this pull request Sep 28, 2020
This switches go-ipfs to dir-index-html after
ipfs/dir-index-html#43
got merged to master
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Broken CID column on DNSLink gateway
2 participants