Skip to content

test: add missing DNSLink -> IPNS resolution#272

Merged
lidel merged 5 commits intomainfrom
feat/dnslink-ipns-test
Feb 24, 2026
Merged

test: add missing DNSLink -> IPNS resolution#272
lidel merged 5 commits intomainfrom
feat/dnslink-ipns-test

Conversation

@lidel
Copy link
Member

@lidel lidel commented Feb 23, 2026

existing DNSLink tests only cover dnslink=/ipfs/, but the dnslink=/ipns/ path is untested. reuse the Ed25519 IPNS record fixture from subdomain_gateway/ to verify the gateway resolves DNSLink -> IPNS -> content correctly.

thank you @aschmahmann for flagging this gap

existing DNSLink tests only cover dnslink=/ipfs/<cid>, but the
dnslink=/ipns/<key> path is untested. reuse the Ed25519 IPNS record
fixture from subdomain_gateway/ to verify the gateway resolves
DNSLink -> IPNS -> content correctly.
@lidel lidel changed the title test: add gateway-conformance test for DNSLink -> IPNS resolution test: add missing DNSLink -> IPNS resolution Feb 23, 2026
@github-actions
Copy link
Contributor

github-actions bot commented Feb 23, 2026

Results against Kubo master:

Summary

Tests Failures Errors Skipped
1250 0 0 0

@github-actions
Copy link
Contributor

github-actions bot commented Feb 23, 2026

Results against Kubo latest:

Summary

Tests Failures Errors Skipped
1250 26 0 0

lidel added a commit to ipfs/kubo that referenced this pull request Feb 23, 2026
point gateway-conformance actions at ipfs/gateway-conformance@14c75521
which adds a conformance test for DNSLink TXT records pointing to
/ipns/<key> instead of /ipfs/<cid>.

ref: ipfs/gateway-conformance#272
lidel added a commit to ipfs/kubo that referenced this pull request Feb 23, 2026
point gateway-conformance actions at ipfs/gateway-conformance@14c75521
which adds a conformance test for DNSLink TXT records pointing to
/ipns/<key> instead of /ipfs/<cid>.

ref: ipfs/gateway-conformance#272
test both original and inlined DNSLink domain forms when the
dnslink TXT record points at /ipns/<key> instead of /ipfs/<cid>.
lidel added a commit to ipfs/kubo that referenced this pull request Feb 23, 2026
point gateway-conformance actions at ipfs/gateway-conformance@14c75521
which adds a conformance test for DNSLink TXT records pointing to
/ipns/<key> instead of /ipfs/<cid>.

ref: ipfs/gateway-conformance#272
@lidel lidel marked this pull request as ready for review February 24, 2026 00:04
test both base58btc peer ID and CIDv1-libp2p-key (base36) forms of
IPNS names in DNSLink TXT records across gateway and subdomain tests.
lidel added a commit to ipfs/kubo that referenced this pull request Feb 24, 2026
point gateway-conformance actions at ipfs/gateway-conformance@14c75521
which adds a conformance test for DNSLink TXT records pointing to
/ipns/<key> instead of /ipfs/<cid>.

ref: ipfs/gateway-conformance#272
Fmt treats triple braces {{{...}}} as escaped literals, not template
vars, causing "too many arguments" panic.
@github-actions
Copy link
Contributor

v0.11.0

Added

  • New test TestDNSLinkGatewayIPNS for DNSLink TXT records pointing at /ipns/<key> instead of /ipfs/<cid>. All prior DNSLink tests only covered the /ipfs/ case. #272
  • New test cases in TestSubdomainGatewayDNSLinkInlining for the subdomain form of DNSLink -> IPNS resolution ({dnslink}.ipns.{gateway} and {inlined-dnslink}.ipns.{gateway}). #272

lidel added a commit to ipfs/kubo that referenced this pull request Feb 24, 2026
point gateway-conformance actions at ipfs/gateway-conformance@14c75521
which adds a conformance test for DNSLink TXT records pointing to
/ipns/<key> instead of /ipfs/<cid>.

ref: ipfs/gateway-conformance#272
@lidel lidel merged commit e2dd3d2 into main Feb 24, 2026
10 checks passed
@lidel lidel deleted the feat/dnslink-ipns-test branch February 24, 2026 00:37
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant