Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(473)

Issue 1128103007: Protected by FLAGS_enable_quic_stateless_reject_support. Add support (Closed)

Created:
5 years, 7 months ago by ramant (doing other things)
Modified:
5 years, 7 months ago
Reviewers:
Ryan Hamilton
CC:
chromium-reviews, cbentzel+watch_chromium.org
Base URL:
https://chromium.googlesource.com/chromium/src.git@Added_support_for_stateless_time_wait_93412303
Target Ref:
refs/pending/heads/master
Project:
chromium
Visibility:
Public.

Description

Protected by FLAGS_enable_quic_stateless_reject_support. Add support for stateless rejects to the QuicDispatcher code. QuicServerSession - Added extra setter, which the dispatcher uses to indicate that session should issue stateless rejects (if supported by the peer). QuicDispatcher - Will use stateless rejects if the session map is over a threshold, configurable via flag. If a stateless reject has been issued, the dispatcher will close the session silently. It will also add the connection-id to the time-wait list, indicating that further packets for the connection-id should be black-holed. Changes to the end_to_end_tests will be made in a separate CL. Merge internal change: 93414515 R=rch@chromium.org

Patch Set 1 #

Unified diffs Side-by-side diffs Delta from patch set Stats (+238 lines, -14 lines) Patch
M net/tools/quic/quic_dispatcher.h View 1 chunk +4 lines, -3 lines 0 comments Download
M net/tools/quic/quic_dispatcher.cc View 6 chunks +32 lines, -4 lines 0 comments Download
M net/tools/quic/quic_dispatcher_test.cc View 6 chunks +181 lines, -7 lines 0 comments Download
M net/tools/quic/quic_server_session.h View 1 chunk +21 lines, -0 lines 0 comments Download

Messages

Total messages: 2 (0 generated)
ramant (doing other things)
5 years, 7 months ago (2015-05-14 23:57:46 UTC) #1
Ryan Hamilton
5 years, 7 months ago (2015-05-15 04:02:39 UTC) #2
lgtm

Powered by Google App Engine
This is Rietveld 408576698