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.

Sign up for GitHub

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

Jump to bottom

strawman for when to close streams #567

Open
wants to merge 3 commits into
base: main
Choose a base branch
from
Open

strawman for when to close streams #567

wants to merge 3 commits into from

Conversation

fluffy
Copy link
Contributor

@fluffy fluffy commented Sep 30, 2024

No description provided.

@fluffy fluffy added the Needs Discussion Tags for issues which need discussion, ideally at an interim or IETF meeting. label Sep 30, 2024
afrind
afrind reviewed Oct 1, 2024
View reviewed changes
Copy link
Collaborator

@afrind afrind left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Individual Comment:

I also made a different proposal here: #442 (comment)

draft-ietf-moq-transport.md Outdated Show resolved Hide resolved
martinduke
martinduke reviewed Oct 1, 2024
View reviewed changes
Copy link
Contributor

@martinduke martinduke left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

As an individual:

I would like to wordsmith something involving stream FIN, but it doesn't have to be in this PR.

draft-ietf-moq-transport.md Outdated Show resolved Hide resolved
draft-ietf-moq-transport.md Outdated Show resolved Hide resolved
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@TimEvens TimEvens TimEvens left review comments

@afrind afrind afrind left review comments

@martinduke martinduke martinduke left review comments

@ianswett ianswett Awaiting requested review from ianswett

At least 1 approving review is required to merge this pull request.

Assignees
No one assigned
Labels
Needs Discussion Tags for issues which need discussion, ideally at an interim or IETF meeting.
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
4 participants
@fluffy @TimEvens @afrind @martinduke