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
Replace suppressSequenceNumber with transport-specific behavior #51
Conversation
I think I have addressed this aspect already in PR #49 |
Correct. I thought Ben's suggestion to tie it to transport made more sense, but I'm not hard over on it. |
Fine for me too. |
draft-ietf-tls-ctls.md
Outdated
the `suppressSequenceNumber` flag in the compression profile being used (see | ||
{{template-based-specialization}}). When this flag is enabled, the S bit in the | ||
configuration octet MUST be cleared. | ||
field entirely. When a reliable transport (e.g., TCP) is in use, the |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
field entirely. When a reliable transport (e.g., TCP) is in use, the | |
field entirely. When a reliable, ordered transport (e.g., TCP) is in use, the |
Co-authored-by: Benjamin M. Schwartz <bemasc@google.com>
{{template-based-specialization}}). When this flag is enabled, the S bit in the | ||
configuration octet MUST be cleared. | ||
field entirely. When a reliable, ordered transport (e.g., TCP) is in use, the | ||
S bit in the configuration octet MUST be cleared and the sequence number |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
S = stream transport ?
S = "sequence number"
…On Tue, Mar 8, 2022 at 3:53 AM Martin Thomson ***@***.***> wrote:
***@***.**** commented on this pull request.
------------------------------
In draft-ietf-tls-ctls.md
<#51 (comment)>
:
> @@ -410,11 +410,10 @@ transports (e.g., TCP), the length field MUST be present. For use over other
transports length information may be inferred from the underlying layer.
Normal DTLS does not provide a mechanism for suppressing the sequence number
-field entirely. In cases where a sequence number is not required (e.g., when a
-reliable transport is in use), a cTLS implementation may suppress it by setting
-the `suppressSequenceNumber` flag in the compression profile being used (see
-{{template-based-specialization}}). When this flag is enabled, the S bit in the
-configuration octet MUST be cleared.
+field entirely. When a reliable, ordered transport (e.g., TCP) is in use, the
+S bit in the configuration octet MUST be cleared and the sequence number
S = stream transport ?
—
Reply to this email directly, view it on GitHub
<#51 (review)>,
or unsubscribe
<https://github.com/notifications/unsubscribe-auth/AAIPLIMKLBNYM7GCFTUTG73U645TPANCNFSM5QD5MIYQ>
.
Triage notifications on the go with GitHub Mobile for iOS
<https://apps.apple.com/app/apple-store/id1477376905?ct=notification-email&mt=8&pt=524675>
or Android
<https://play.google.com/store/apps/details?id=com.github.android&referrer=utm_campaign%3Dnotification-email%26utm_medium%3Demail%26utm_source%3Dgithub>.
You are receiving this because you modified the open/close state.Message
ID: ***@***.***>
|
I'm slightly concerned about the fact that we have two different wire behaviors with the same transcript, but because the integrity tag includes the header, I think it's OK.
@martinthomson would welcome your thoughts.