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

Support for STARTTLS #35

Closed
gloinul opened this issue Feb 22, 2021 · 5 comments
Closed

Support for STARTTLS #35

gloinul opened this issue Feb 22, 2021 · 5 comments

Comments

@gloinul
Copy link
Owner

gloinul commented Feb 22, 2021

So part of the discussion in #27 brought up the case of STARTTLS. I think that needs more consideration as due to even RFC 6083 requirement to have SCTP-AUTH, one can't move from a plain SCTP association to one with DTLS. Instead one actually need to create an association with the intent to use DTLS. So are there any point in attempting to support STARTTLS like for services that use TLS/TCP where it works.

@tuexen
Copy link
Contributor

tuexen commented Feb 22, 2021

I only know about RFC 3788, which requires setting up the SCTP association appropriately, then exchanges the START-TLS and START-TLS-ACK message in the clear and then do TLS. RFC 6083 did not exist at that time.

The reason RFC 3788 was written is that there were some SIGTRAN specifications already published and they only specified the unsecured protocol. We requested separate port numbers for the protocols over TLS, but IANA declined that. So we had to do the START-TLS dance.

To be honest, in my limited testing I have done for a German mobile operator, I have never seen SIGTRAN over TLS... So I don't have a problem with not supporting START-TLS, we just need to be clear about it.

@gloinul
Copy link
Owner Author

gloinul commented May 10, 2021

I think the way forward here is to explicitly note that STARTTLS will not work with this specification.

@teiclap
Copy link
Collaborator

teiclap commented Jun 17, 2021

The note that STARTTLS will not work with this specification has been added.

@emanjon
Copy link
Collaborator

emanjon commented Jun 21, 2021

The way forward and the added note looks good.

@gloinul
Copy link
Owner Author

gloinul commented Jun 22, 2021

The note has been reformulated in the PR due to that it was contextual problematic. We may want to further improve the style of it in the future.

@gloinul gloinul closed this as completed Jul 1, 2021
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

No branches or pull requests

4 participants