[go: nahoru, domu]

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

Server push, client side #341

Closed
wants to merge 4 commits into from
Closed

Conversation

ddragana
Copy link
Contributor
@ddragana ddragana commented Dec 6, 2019

No description provided.

    Http3Connection only handles common elements:
     - holds control and qpack streams,
     - holds a HashMap of active transactions,
     - holds the Http3State of a http3 connection,
     - performs sending data for control, qpack and transaction streams,
     - handles decoding of new unidi streams(the client/server handlers differ in how they handle a  Push streams, therefore handle_new_unidi_stream returns if a stream was a push stream and the client/server handler consume them appropriately),
     - handles streams reading (this function may return that a stream is a push stream or a list of control frames to be handle by the client/server handler)

    The client/server handlers:
     - holds a base_handler(Http3Connection)
     - http3 events
     - neqo_transport::Connection (this is only for the client)
     - process_htttp3 has been moved from Http3Connection to the client/server handlers,
     - also check_connection_events which iterates through ConnectionEvents and calls handlers for the events (handlers are implemented in Http3Connection or in the client/server handlers)
@ddragana
Copy link
Contributor Author
ddragana commented Dec 6, 2019

This needs to wait for #334 .

@ddragana
Copy link
Contributor Author
ddragana commented Dec 6, 2019

This needs rebase, the failure is unrelated to this pr.

@ddragana
Copy link
Contributor Author

I made a new rebased branch, closing this pr

@ddragana ddragana closed this Dec 17, 2019
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

1 participant