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

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

Merged
jessicaschilling merged 2 commits intomasterfrom
fix/links-when-dnslink
Sep 28, 2020
Merged

feat: improved CID column (filename and dnslink support)#43
jessicaschilling merged 2 commits intomasterfrom
fix/links-when-dnslink

Conversation

@lidel
Copy link
Copy Markdown
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 lidel force-pushed the fix/links-when-dnslink branch from ea4e54a to b12c35a Compare September 28, 2020 13:02
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
Copy Markdown
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