Enable comments when loaded via http://localhost:8080/ipns/blog.ipfs.io/ #43

Closed
opened 2026-10-05 05:11:00 +02:00 by adam · 5 comments
Owner

Right now comments do not load for users of IPFS Companion due to DNSLink redirect.
I think we should at least whitelist the default local gateway, it will probably fix it for most users.

Not sure who has the admin on http://discuss.ipfs.io/, but AFAIK we just need to whitelist http://localhost:8080/ipns/blog.ipfs.io/ http://blog.ipfs.io.ipns.localhost:8080 as a valid embed URL.

Right now comments do not load for users of IPFS Companion due to DNSLink redirect. I think we should at least whitelist the default local gateway, it will probably fix it for most users. Not sure who has the admin on http://discuss.ipfs.io/, but AFAIK we just need to whitelist <del> `http://localhost:8080/ipns/blog.ipfs.io/` </del> `http://blog.ipfs.io.ipns.localhost:8080` as a valid embed URL. <!-- github-sync-id:828178246 -->
adam closed this issue 2026-10-05 05:11:00 +02:00
Author
Owner

@lidel Added these, as well as the Fleek preview domains that the new blog will be hosted on, in:

Note that this does NOT solve for display on the legacy blog, as the site code itself would need to be changed, and relaunch of the new blog is imminent. Please note as well that we have verification of comments functionality as part of pre-launch checklist https://github.com/ipfs/ipfs-blog/issues/22, so there may be additional amends in future to get this up and running perfectly.

@lidel Added these, as well as the Fleek preview domains that the new blog will be hosted on, in: - https://github.com/ipfs/ipfs-blog/pull/83 - Discourse whitelist Note that this does NOT solve for display on the legacy blog, as the site code itself would need to be changed, and relaunch of the new blog is imminent. Please note as well that we have verification of comments functionality as part of pre-launch checklist https://github.com/ipfs/ipfs-blog/issues/22, so there may be additional amends in future to get this up and running perfectly. <!-- github-comment-id:796187183 -->
Author
Owner

Re-opening for further troubleshooting as first batch of post-launch work.

Re-opening for further troubleshooting as first batch of post-launch work. <!-- github-comment-id:816098304 -->
Author
Owner

@lidel Am afraid that approach didn't work 😟 Do you have time to troubleshoot this with @zebateira and/or @jdiogopeixoto at some point?

@lidel Am afraid that approach didn't work 😟 Do you have time to troubleshoot this with @zebateira and/or @jdiogopeixoto at some point? <!-- github-comment-id:816114757 -->
Author
Owner

Re-opening this issue until we can confirm that #120 fixed this.

Re-opening this issue until we can confirm that #120 fixed this. <!-- github-comment-id:816241156 -->
Author
Owner

I confirm, this is fixed on the local origin (http://blog.ipfs.io.ipns.localhost:*)

I confirm, this is fixed on the local origin (`http://blog.ipfs.io.ipns.localhost:*`) <!-- github-comment-id:816663879 -->
Sign in to join this conversation.
No labels
1 Participants
Notifications
Due Date
No due date set.
Dependencies

No dependencies set.

Reference: adam/ipfs-blog#43