Re: [quicwg/base-drafts] Clarify Connection ID and Version number endianness (#786)

Subodh Iyengar <notifications@github.com> Fri, 22 September 2017 20:02 UTC

Return-Path: <noreply@github.com>
X-Original-To: quic-issues@ietfa.amsl.com
Delivered-To: quic-issues@ietfa.amsl.com
Received: from localhost (localhost [127.0.0.1]) by ietfa.amsl.com (Postfix) with ESMTP id 5931F1326DF for <quic-issues@ietfa.amsl.com>; Fri, 22 Sep 2017 13:02:12 -0700 (PDT)
X-Virus-Scanned: amavisd-new at amsl.com
X-Spam-Flag: NO
X-Spam-Score: -8.396
X-Spam-Level:
X-Spam-Status: No, score=-8.396 tagged_above=-999 required=5 tests=[BAYES_00=-1.9, DKIM_SIGNED=0.1, DKIM_VALID=-0.1, DKIM_VALID_AU=-0.1, HTML_IMAGE_ONLY_28=1.404, HTML_MESSAGE=0.001, RCVD_IN_DNSWL_HI=-5, RCVD_IN_MSPIKE_H2=-2.8, SPF_PASS=-0.001] autolearn=ham autolearn_force=no
Authentication-Results: ietfa.amsl.com (amavisd-new); dkim=pass (1024-bit key) header.d=github.com
Received: from mail.ietf.org ([4.31.198.44]) by localhost (ietfa.amsl.com [127.0.0.1]) (amavisd-new, port 10024) with ESMTP id efYtWi0osVyD for <quic-issues@ietfa.amsl.com>; Fri, 22 Sep 2017 13:02:10 -0700 (PDT)
Received: from github-smtp2b-ext-cp1-prd.iad.github.net (github-smtp2-ext2.iad.github.net [192.30.252.193]) (using TLSv1.2 with cipher AECDH-AES256-SHA (256/256 bits)) (No client certificate requested) by ietfa.amsl.com (Postfix) with ESMTPS id A56F8129B7A for <quic-issues@ietf.org>; Fri, 22 Sep 2017 13:02:10 -0700 (PDT)
Date: Fri, 22 Sep 2017 13:02:09 -0700
DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=github.com; s=pf2014; t=1506110529; bh=kD122Ru+1hqjJ9dbh6Ypp3IA/vluxkU2xAm/fctfo0M=; h=From:Reply-To:To:Cc:In-Reply-To:References:Subject:List-ID: List-Archive:List-Post:List-Unsubscribe:From; b=VV9Qy49cq8yT0/TShNBwjnRFF5xCt3DV03dzwN/6jnG6AvBk0oG6WmlrzkN1X58j3 5jha929pxTFyF90F85WRpwaMHjEAYuNgNolhivhZHSro0oEBf1rGIlSrxVvhjMqgCS VwTuCJWe4zOI8WDaIULh4hm6ISE5wdEFqKaEDm+k=
From: Subodh Iyengar <notifications@github.com>
Reply-To: quicwg/base-drafts <reply+0166e4ab94e75a5b4d1e4779fff7226840c746371733fca292cf0000000115dd2e4192a169ce0f78bdb5@reply.github.com>
To: quicwg/base-drafts <base-drafts@noreply.github.com>
Cc: Subscribed <subscribed@noreply.github.com>
Message-ID: <quicwg/base-drafts/issues/786/331547058@github.com>
In-Reply-To: <quicwg/base-drafts/issues/786@github.com>
References: <quicwg/base-drafts/issues/786@github.com>
Subject: Re: [quicwg/base-drafts] Clarify Connection ID and Version number endianness (#786)
Mime-Version: 1.0
Content-Type: multipart/alternative; boundary="--==_mimepart_59c56c41e6e7e_73db3fcada084f781050b7"; charset="UTF-8"
Content-Transfer-Encoding: 7bit
Precedence: list
X-GitHub-Sender: siyengar
X-GitHub-Recipient: quic-issues
X-GitHub-Reason: subscribed
X-Auto-Response-Suppress: All
X-GitHub-Recipient-Address: quic-issues@ietf.org
Archived-At: <https://mailarchive.ietf.org/arch/msg/quic-issues/lKZJLe86YhQTMpPAUVj1tPSPpMs>
X-BeenThere: quic-issues@ietf.org
X-Mailman-Version: 2.1.22
List-Id: Notification list for GitHub issues related to the QUIC WG <quic-issues.ietf.org>
List-Unsubscribe: <https://www.ietf.org/mailman/options/quic-issues>, <mailto:quic-issues-request@ietf.org?subject=unsubscribe>
List-Archive: <https://mailarchive.ietf.org/arch/browse/quic-issues/>
List-Post: <mailto:quic-issues@ietf.org>
List-Help: <mailto:quic-issues-request@ietf.org?subject=help>
List-Subscribe: <https://www.ietf.org/mailman/listinfo/quic-issues>, <mailto:quic-issues-request@ietf.org?subject=subscribe>
X-List-Received-Date: Fri, 22 Sep 2017 20:02:12 -0000

It seems fairly weird to treat conn ids as 64 bit ints because load balancers are going to do bit twiddling on the connection id to either encrypt / store state in the connid. I can see the argument of being consistent with other fields, but it still seems like it will cause some strange bugs if we specify endianness for things that are not actually ints. My preference is to specifically treat connid as not an int.

-- 
You are receiving this because you are subscribed to this thread.
Reply to this email directly or view it on GitHub:
https://github.com/quicwg/base-drafts/issues/786#issuecomment-331547058