Re: [xmpp] draft-cridland-xmpp-session-00

Ralph Meijer <ralphm@ik.nu> Tue, 10 June 2014 08:26 UTC

Return-Path: <ralphm@ik.nu>
X-Original-To: xmpp@ietfa.amsl.com
Delivered-To: xmpp@ietfa.amsl.com
Received: from localhost (ietfa.amsl.com [127.0.0.1]) by ietfa.amsl.com (Postfix) with ESMTP id 386891A020B for <xmpp@ietfa.amsl.com>; Tue, 10 Jun 2014 01:26:31 -0700 (PDT)
X-Virus-Scanned: amavisd-new at amsl.com
X-Spam-Flag: NO
X-Spam-Score: -2.551
X-Spam-Level:
X-Spam-Status: No, score=-2.551 tagged_above=-999 required=5 tests=[BAYES_00=-1.9, RP_MATCHES_RCVD=-0.651] autolearn=ham
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 IEwQBx5CqDid for <xmpp@ietfa.amsl.com>; Tue, 10 Jun 2014 01:26:29 -0700 (PDT)
Received: from mag.ik.nu (mag.ik.nu [IPv6:2001:16f8:4::61]) (using TLSv1.2 with cipher AECDH-AES256-SHA (256/256 bits)) (No client certificate requested) by ietfa.amsl.com (Postfix) with ESMTPS id 085261A02D9 for <xmpp@ietf.org>; Tue, 10 Jun 2014 01:26:29 -0700 (PDT)
Received: from mag.ik.nu (localhost [127.0.0.1]) by mag.ik.nu (Postfix) with ESMTP id CA0CFA1021 for <xmpp@ietf.org>; Tue, 10 Jun 2014 10:26:26 +0200 (CEST)
X-Virus-Scanned: amavisd-new at ik.nu
Received: from mag.ik.nu ([127.0.0.1]) by mag.ik.nu (mag.ik.nu [127.0.0.1]) (amavisd-new, port 10024) with SMTP id WiD6p01WxrDw for <xmpp@ietf.org>; Tue, 10 Jun 2014 10:26:26 +0200 (CEST)
Received: from [192.168.3.215] (s53751670.adsl.online.nl [83.117.22.112]) by mag.ik.nu (Postfix) with ESMTPSA id F3176A100F for <xmpp@ietf.org>; Tue, 10 Jun 2014 10:26:25 +0200 (CEST)
Message-ID: <5396C131.8030508@ik.nu>
Date: Tue, 10 Jun 2014 10:26:25 +0200
From: Ralph Meijer <ralphm@ik.nu>
User-Agent: Mozilla/5.0 (X11; Linux x86_64; rv:24.0) Gecko/20100101 Thunderbird/24.5.0
MIME-Version: 1.0
To: xmpp@ietf.org
References: <CAKHUCzwJrykJrOscQowXOKZY1Aq7MA+YRWz=XanDknY+7zq6qg@mail.gmail.com> <B97418EC-47DF-439E-85C2-835761F6D694@andyet.net> <5395DF40.2030509@stpeter.im> <292F40A9-A302-477B-AF26-57B1D3024BEC@mumbo.ca> <CAKHUCzyoB04UM63afZctwsCTRKCs=WJ_DjSZrS4Vw8w3iqUarg@mail.gmail.com> <557B118B-21BE-43FD-905A-9B725836E66F@mumbo.ca> <CAKHUCzyamFr6LAk0B+fkdvFg7hoapakNj0bJ9yKPFTd3sET52Q@mail.gmail.com> <CAOb_FnzePrYr++b8r2oCS07eLCB7R0kuFmY2wkqZB=M8SEP0Vw@mail.gmail.com>
In-Reply-To: <CAOb_FnzePrYr++b8r2oCS07eLCB7R0kuFmY2wkqZB=M8SEP0Vw@mail.gmail.com>
X-Enigmail-Version: 1.6
Content-Type: text/plain; charset=us-ascii
Content-Transfer-Encoding: 7bit
Archived-At: http://mailarchive.ietf.org/arch/msg/xmpp/zFZF303NsyEKSVv-3GkCmDeapZ0
Subject: Re: [xmpp] draft-cridland-xmpp-session-00
X-BeenThere: xmpp@ietf.org
X-Mailman-Version: 2.1.15
Precedence: list
List-Id: XMPP Working Group <xmpp.ietf.org>
List-Unsubscribe: <https://www.ietf.org/mailman/options/xmpp>, <mailto:xmpp-request@ietf.org?subject=unsubscribe>
List-Archive: <http://www.ietf.org/mail-archive/web/xmpp/>
List-Post: <mailto:xmpp@ietf.org>
List-Help: <mailto:xmpp-request@ietf.org?subject=help>
List-Subscribe: <https://www.ietf.org/mailman/listinfo/xmpp>, <mailto:xmpp-request@ietf.org?subject=subscribe>
X-List-Received-Date: Tue, 10 Jun 2014 08:26:31 -0000

On 2014-06-10 10:07, Kevin Smith wrote:
> On Tue, Jun 10, 2014 at 8:43 AM, Dave Cridland <dave@cridland.net> wrote:
>>> This draft will require servers and client changes, you could accomplish
>>> the same goal by a pure informational draft pointing such features are
>>> optional. Then only certain clients need to change. Note: Good clients like
>>> Swift already ignore the session feature.
>>>
>>
>> Then it's not a good client - the session feature, if advertised, is
>> mandatory.
> 
> Yeah, I don't believe this is true. Swift treats the session start as
> unnecessary, but if it's offered by the server it'll negotiate it. The
> relevant code is splattered around
> http://swift.im/git/swift/tree/Swiften/Client/ClientSession.cpp

Oh, but it is. RFC 3921 says, section 3 says:

    Upon being so informed that session establishment is required [*]
    (and after completing resource binding), the client MUST establish
    a session [..]

[*] The part above that shows the stream feature being advertised, and
because of that wording I believe it implies that advertising Session
Establishment makes it required.

Then RFC 6121 only mentions that the protocol is unnecessary (Appendix
E), but doesn't explicitly make it optional when advertised. I.e. it
doesn't change the protocol, just doesn't document it any more.


>> So if you remove the <optional/> marker from M-Link, every
>> conforming client has to negotiate it.
> 
> Every 3920/1 client, that is, rather than every 6120/1 client. yes?

I think you will find there are no pure forms of either, in reality.


>> You can't claim that if it's RFC 6121
>> only then it's exempt, because then certain servers won't work (I think
>> ejabberd is one that actually requires the <session/>, in line with RFC
>> 3921).
> 
> Right. Clients still need to implement this for old servers (I assume
> modern ejabberd /doesn't/ require this, but very old versions are
> undoubtedly out in the wild).

I disagree. Clients only need to implement this if they want to benefit
from the removal of a roundtrip with servers advertising this flag.


> I think the draft is roughly the right thing to do. Nits:
> 
> od->of
> 
> <optional/> really isn't what this really is. Is there scope for
> naming it <obsolete/>? How widely deployed are clients-servers that
> use optional and are unlikely to be upgradable? I'm uncomfortable with
> standardising that <optional/> means MUST NOT.

As I said before, adding this flag makes the client *choose* to not
negotiate Session Establishment, therefore making it optional. I don't
really care for renaming this with existing implementations (like your
own) already having this deployed in the field. I don't think it does
anyone a favour, except protocol purists. I do agree <optional/> should
mean 'SHOULD NOT negotiate'.


> If we have to do this, we should probably add some text that
> <optional/> is only used in the context of session startup.

I'm not sure what you mean here. To be sure, this flag is only being
defined for this namespace / stream feature.


-- 
ralphm