lib/mvfst
1
0
Fork 0
mirror of https://github.com/facebookincubator/mvfst.git synced 2025-08-08 09:42:06 +03:00
Code Activity

use uint32 for max Datagram size

Browse Source 
Reviewed By: hanidamlaj

Differential Revision: D44080220

fbshipit-source-id: e84c2e3718f7d06fd245770855c8cdc05f49bd86
This commit is contained in:
Luca Niccolini
2023-03-15 02:02:59 -07:00
committed by Facebook GitHub Bot
parent d6f02bda04
commit 93096fc8b9
2 changed files with 6 additions and 3 deletions
Download Patch File Download Diff File Expand all files Collapse all files

5
quic/api/test/QuicTransportBaseTest.cpp
View File

@@ -538,7 +538,10 @@ class TestQuicTransport
} }
void enableDatagram() { void enableDatagram() {
conn_->datagramState.maxReadFrameSize = 65535; // Note: the RFC says to use 65535 to enable the datagram extension.
// We are using +1 in tests to make sure that we avoid representing this
// value with an uint16
conn_->datagramState.maxReadFrameSize = 65536;
conn_->datagramState.maxReadBufferSize = 10; conn_->datagramState.maxReadBufferSize = 10;
} }

4
quic/state/StateData.h
View File

@@ -695,8 +695,8 @@ struct QuicConnectionStateBase : public folly::DelayedDestruction {
uint64_t numProbesWritableBytesLimited{0}; uint64_t numProbesWritableBytesLimited{0};
struct DatagramState { struct DatagramState {
uint16_t maxReadFrameSize{kDefaultMaxDatagramFrameSize}; uint32_t maxReadFrameSize{kDefaultMaxDatagramFrameSize};
uint16_t maxWriteFrameSize{kDefaultMaxDatagramFrameSize}; uint32_t maxWriteFrameSize{kDefaultMaxDatagramFrameSize};
uint32_t maxReadBufferSize{kDefaultMaxDatagramsBuffered}; uint32_t maxReadBufferSize{kDefaultMaxDatagramsBuffered};
uint32_t maxWriteBufferSize{kDefaultMaxDatagramsBuffered}; uint32_t maxWriteBufferSize{kDefaultMaxDatagramsBuffered};
// Buffers Incoming Datagrams // Buffers Incoming Datagrams