Skip to content

Commit

Permalink
Use draft29 as default quic version as v1 is not released yet (java-n…
Browse files Browse the repository at this point in the history
…ative-access#279)

Motivation:

We should not use v1 as default version as it was not released yet.

Modifications:

Use draft29 as default quic version

Result:

Use a "released" quic version by default
  • Loading branch information
normanmaurer authored May 12, 2021
1 parent 18014a8 commit 2944eac
Showing 1 changed file with 4 additions and 1 deletion.
Original file line number Diff line number Diff line change
Expand Up @@ -54,7 +54,10 @@ public abstract class QuicCodecBuilder<B extends QuicCodecBuilder<B>> {

QuicCodecBuilder(boolean server) {
Quic.ensureAvailability();
this.version = Quiche.QUICHE_PROTOCOL_VERSION;
// Use draft29 for now until v1 is really ready to use by default.
//
// See https://mailarchive.ietf.org/arch/msg/quic/i7CdtRA-iskuTftpEpMXMrB0FSY/
this.version = 0xff00_001d;
this.localConnIdLength = Quiche.QUICHE_MAX_CONN_ID_LEN;
this.server = server;
}
Expand Down

0 comments on commit 2944eac

Please sign in to comment.