gr1ffon/quic-go
1
0
Fork 0
forked from quic-go/quic-go
Code Pull Requests Activity

don't write any timestamps in QUIC 34 ACK frames

Browse Source 
ref #182
This commit is contained in:
Marten Seemann
2016-06-21 18:14:30 +07:00
parent 193f0dd793
commit a92bd2bf55
1 changed files with 2 additions and 4 deletions
Download Patch File Download Diff File Expand all files Collapse all files

6
frames/ack_frame_new.go
View File

@@ -257,9 +257,7 @@ func (f *AckFrameNew) Write(b *bytes.Buffer, version protocol.VersionNumber) err
return errors.New("BUG: Inconsistent number of ACK ranges written")
}
b.WriteByte(0x01) // Just one timestamp
b.WriteByte(0x00) // Delta Largest observed
utils.WriteUint32(b, 0) // First timestamp
b.WriteByte(0) // no timestamps
return nil
}
@@ -267,7 +265,7 @@ func (f *AckFrameNew) Write(b *bytes.Buffer, version protocol.VersionNumber) err
// MinLength of a written frame
func (f *AckFrameNew) MinLength(version protocol.VersionNumber) (protocol.ByteCount, error) {
var length protocol.ByteCount
length = 1 + 2 + 1 + 1 + 4 // 1 TypeByte, 2 ACK delay time, 1 Num Timestamp, 1 Delta Largest Observed, 4 FirstTimestamp
length = 1 + 2 + 1 // 1 TypeByte, 2 ACK delay time, 1 Num Timestamp
length += protocol.ByteCount(protocol.GetPacketNumberLength(f.LargestObserved))
missingSequenceNumberDeltaLen := protocol.ByteCount(protocol.PacketNumberLen6)