Skip to content

feat(/): allow forward slash in domain segments - #48

Open
msimerson wants to merge 1 commit into
hapijs:masterfrom
msimerson:forward-slash
Open

msimerson wants to merge 1 commit into
hapijs:masterfrom
msimerson:forward-slash

Conversation

@msimerson

Copy link
Copy Markdown
Contributor

used in Classless IN-ADDR.ARPA delegation RFC 2317

used in Classless IN-ADDR.ARPA delegation (RFC 2317)
@Marsup

Marsup commented Sep 16, 2026

Copy link
Copy Markdown
Collaborator

I've been looking into this matter because I'm unfamiliar with this RFC, and I'm not sure it belongs in address, or at least maybe not in domains. If I'm not mistaken, this is about DNS zone specifications. While domains are closely related, I think validating DNS zones has a lot more differences than just that slash. It seems super rarely used, wouldn't you be better off with a small joi extension to deal with your use case?

@msimerson

Copy link
Copy Markdown
Contributor Author

Your assessment is correct, which is why I defaulted it to off. I'm already using another option so I understand if this doesn't exactly fit this module.

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