-
Notifications
You must be signed in to change notification settings - Fork 51
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
Federation client shouldn't follow CNAME records in SRV records #127
Comments
this turns out to be surprisingly hard to do correctly in Go, so I'm not sure this is worth fixing. It's pretty harmless if we are a bit too tolerant. |
I know it forbids it, but everytime a server lets me specify a CNAME nonetheless, I dance a happy dance. Even if not accordingly strictly to spec, it is not wrong to be a bit lenient, so I propose to not do anything about this and close the issue :-) |
I fundamentally disagree with past-me here. Given the question "should CNAMEs be accepted as the target of SRV records", there should be one answer that applies to the whole of the matrix ecosystem, otherwise we'll be in the frustrating situation of connections working intermittently. (eg, you'll be able to federate with servers written in Go, but not in Python) Before completely deciding what to do here, I'd be interested to know what Synapse does, but given, as Brendan says, CNAMEs are not normally valid as the target of SRV records, I am inclined to say they should not be accepted. Related: matrix-org/matrix-spec#606 |
RFC2782 forbids CNAME records as SRV record targets but we need to check it because Go completely ignores that
The text was updated successfully, but these errors were encountered: