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

Error states missing #4277

Closed
larseggert opened this issue Oct 28, 2020 · 3 comments
Closed

Error states missing #4277

larseggert opened this issue Oct 28, 2020 · 3 comments
Labels
-transport ietf-lc An issue that was raised during IETF Last Call.

Comments

@larseggert
Copy link
Member

In looking at "Figure 2: States for Sending Parts of Streams" it occurred
to me that this was the major states but there were possibly a number of error
states missing. It would be useful to clarify.

@larseggert larseggert added -transport ietf-lc An issue that was raised during IETF Last Call. labels Oct 28, 2020
@larseggert larseggert added this to the transport-genart milestone Oct 28, 2020
@MikeBishop
Copy link
Contributor

Errors result in closing the connection, at which point the streams no longer have a state.

@martinthomson
Copy link
Member

I think we'd need something more concrete for this to be actionable. As this state machine has been implemented a number of times now, I don't believe that we've missed any states.

And, as Mike says, many error states are not relevant as genuine errors result in the connection closing. Of course, there are far fewer concrete errors than you might first think.

@martinthomson martinthomson added the proposal-ready An issue which has a proposal that is believed to be ready for a consensus call. label Nov 4, 2020
@LPardue
Copy link
Member

LPardue commented Dec 8, 2020

The proposed resolution was to close to with action, which was signalled to the appropriate review channel.

Hearing no pushback, I'm closing this.

@LPardue LPardue closed this as completed Dec 8, 2020
@LPardue LPardue removed the proposal-ready An issue which has a proposal that is believed to be ready for a consensus call. label Dec 10, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
-transport ietf-lc An issue that was raised during IETF Last Call.
Projects
None yet
Development

No branches or pull requests

4 participants