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

fix wrong stream type used in sending state stream secion #4169

Closed
wants to merge 1 commit into from

Conversation

VinozzZ
Copy link

@VinozzZ VinozzZ commented Sep 27, 2020

type 0 is for a client initiated stream and type 1 is for a server initiated stream

@VinozzZ VinozzZ changed the title Wrong stream type used in sending state stream secion fix wrong stream type used in sending state stream secion Sep 27, 2020
@MikeBishop
Copy link
Contributor

This is not incorrect. Stream 0 is client-initiated, which means that it is a "stream initiated by a peer" for servers. Stream 1 is server-initiated, which means that it is a "stream initiated by a peer" for clients.

@VinozzZ
Copy link
Author

VinozzZ commented Sep 28, 2020

This is not incorrect. Stream 0 is client-initiated, which means that it is a "stream initiated by a peer" for servers. Stream 1 is server-initiated, which means that it is a "stream initiated by a peer" for clients.

A-ha, I see. That makes more sense. Thank you for the clarification

@VinozzZ VinozzZ closed this Sep 28, 2020
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

Successfully merging this pull request may close these issues.

None yet

2 participants