Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Discovery TXT record contents #103

Open
TheEnbyperor opened this issue Jul 31, 2024 · 2 comments
Open

Discovery TXT record contents #103

TheEnbyperor opened this issue Jul 31, 2024 · 2 comments
Labels

Comments

@TheEnbyperor
Copy link

It is currently unclear in the specification if the contents of the _domainconnect TXT record is a FQDN, or a partial URI. That is can the record contain a path element; example.net/connect being valid as well as example.net.

From poking at iCloud's implementation they seem to treat it as partial URI and not an FQDN. I'm unsure what other current implementations do.

As for my opinion on this I think it should be a partial URI, and not just an FQDN.

@pawel-kow
Copy link
Member

Excellent question!
The specification is obviously not specific enough about it. I would rather opt for partial URI interpretation - this is at least what can be derived from the text how the URL to /settings REST API is built.
The fact that the spec is now silent about it and all examples actually use FQDN may be indeed misleading.

@greenreader9
Copy link

I agree with @TheEnbyperor, in that a FQDN should not be required, but a partial URI should be allowed.

Since the service provider just makes a request to the endpoint suffixed with the required paths and parameters, the DNS provider should be allowed to do their own versioning within paths (such as "dnsprovider.tld/api/v1/domainConnect/".

As long as a valid URI is provided, I don't see a reason why paths should not be allowed.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

No branches or pull requests

3 participants