JSON-Schema-Test-Suite icon indicating copy to clipboard operation
JSON-Schema-Test-Suite copied to clipboard

More hostname label separator tests

Open davishmcclurg opened this issue 5 months ago • 2 comments
trafficstars

My implementation doesn't yet support all idn-hostname label separators properly, but (for multiple reasons) it is still passing the tests added here: https://github.com/json-schema-org/JSON-Schema-Test-Suite/pull/760

These are the tests I came up with while fixing/reproducing the issues.

davishmcclurg avatar Jun 16 '25 00:06 davishmcclurg

A lot of these fail in my implementation (they show as valid where the test says it's invalid), but that likely just means I need to mark these as TODO, due to a gap in the library I'm using.

karenetheridge avatar Jun 16 '25 01:06 karenetheridge

Looking at the IDNA specifications some more, it's unclear if the extended set of label separators is included in RFC 5890. The only reference I can find is in an appendix of RFC 5891 ("Summary of Major Changes from IDNA2003" (RFC 3490)):

Remove the dot separator from the mandatory part of the protocol.

Not sure what that means.

They're in RFC 3490 (as noted in #760), but I don't know how relevant that is since JSON schema uses 5890 for idn-hostname.

They're also included in Unicode technical standard 46, which references both IDNA versions. 🤷

Leading/trailing separators are similarly confusing.

davishmcclurg avatar Jun 16 '25 16:06 davishmcclurg