Skip to content

[snrc resolver] report expiry and availability correctly - #1861

Closed
brenzi wants to merge 10 commits into
ab/resolver-labelhash-queryfrom
ab/snrc-resolver-expiry-and-availability
Closed

[snrc resolver] report expiry and availability correctly#1861
brenzi wants to merge 10 commits into
ab/resolver-labelhash-queryfrom
ab/snrc-resolver-expiry-and-availability

Conversation

@brenzi

@brenzi brenzi commented Sep 3, 2026

Copy link
Copy Markdown
Collaborator

Problem with #1860 alone: It prepares the resolver to be used to query availability of names without revealing them, but the response can be misleading because we don't handle expiry and reserved status meaningfully.

  • An expired name beyond its grace period would still resolve although it is actually available again.
  • Reserved names would show as available although they aren't
  • taken-but-resolverless name 404s identically to a free one

This PR fixes the problems above

  • report expiry and reserved status (preparing for reporting a specific reason)
  • introduce meaningful http error codes and handle them in Names.hs

@brenzi brenzi changed the title report expiry and availability correctly [snrc resolver] report expiry and availability correctly Sep 3, 2026
@brenzi
brenzi force-pushed the ab/snrc-resolver-expiry-and-availability branch from 4b5c7bc to e4e3d0d Compare September 4, 2026 06:51
@brenzi
brenzi marked this pull request as ready for review September 4, 2026 07:17
shumvgolove and others added 4 commits September 4, 2026 14:52
# Conflicts:
#	scripts/resolver/README.md
#	scripts/resolver/service/snrc-resolve.py
#	scripts/resolver/service/test_snrc_resolve.py
@brenzi
brenzi removed this pull request from stack #1862 September 10, 2026 11:29
@brenzi

brenzi commented Sep 10, 2026

Copy link
Copy Markdown
Collaborator Author

superseded by #1863, which now carries this work against master

@brenzi brenzi closed this Sep 10, 2026
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.

2 participants