Re: [apps-discuss] APPSDIR review of draft-ietf-simple-chat-13
Alexey Melnikov <alexey.melnikov@isode.com> Wed, 08 February 2012 18:17 UTC
Return-Path: <alexey.melnikov@isode.com>
X-Original-To: apps-discuss@ietfa.amsl.com
Delivered-To: apps-discuss@ietfa.amsl.com
Received: from localhost (localhost [127.0.0.1]) by ietfa.amsl.com (Postfix) with ESMTP id 3E46621E8010; Wed, 8 Feb 2012 10:17:15 -0800 (PST)
X-Virus-Scanned: amavisd-new at amsl.com
X-Spam-Flag: NO
X-Spam-Score: -102.177
X-Spam-Level:
X-Spam-Status: No, score=-102.177 tagged_above=-999 required=5 tests=[AWL=0.422, BAYES_00=-2.599, USER_IN_WHITELIST=-100]
Received: from mail.ietf.org ([12.22.58.30]) by localhost (ietfa.amsl.com [127.0.0.1]) (amavisd-new, port 10024) with ESMTP id fyRuPTmhsxqG; Wed, 8 Feb 2012 10:17:14 -0800 (PST)
Received: from rufus.isode.com (cl-125.lon-03.gb.sixxs.net [IPv6:2a00:14f0:e000:7c::2]) by ietfa.amsl.com (Postfix) with ESMTP id EFB0321E8019; Wed, 8 Feb 2012 10:17:12 -0800 (PST)
DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; t=1328725031; d=isode.com; s=selector; i=@isode.com; bh=22OsjMD9vGKcp0leXaleJwVilw9Q0VYbCTNhd0ZVVco=; h=From:Sender:Reply-To:Subject:Date:Message-ID:To:Cc:MIME-Version: In-Reply-To:References:Content-Type:Content-Transfer-Encoding: Content-ID:Content-Description; b=C44bIXAMWnSK5opMpeaSu8kfjR6z9dDpSSa7qEilw+y1bV96h3f27NnlFTTuqjV8+ooQ0a 5LTFJwvA8mcHfaiTocZlua5zMd4Ek2Zn9sfH2xf7I1ikI1MhtXyStKFkxmCqtcqcRIDJgw snIC5RipyLudGmDOA3ogQpSrqxkQbDo=;
Received: from [172.16.1.29] (shiny.isode.com [62.3.217.250]) by rufus.isode.com (submission channel) via TCP with ESMTPSA id <TzK8JQAvKLyX@rufus.isode.com>; Wed, 8 Feb 2012 18:17:10 +0000
Message-ID: <4F32BC2A.9040503@isode.com>
Date: Wed, 08 Feb 2012 18:17:14 +0000
From: Alexey Melnikov <alexey.melnikov@isode.com>
User-Agent: Mozilla/5.0 (Windows NT 6.1; rv:9.0) Gecko/20111222 Thunderbird/9.0.1
To: "Miguel A. Garcia" <Miguel.A.Garcia@ericsson.com>
References: <4F2FA266.8040406@telecomitalia.it> <4F313153.8050806@ericsson.com>
In-Reply-To: <4F313153.8050806@ericsson.com>
MIME-Version: 1.0
Content-Type: text/plain; charset="ISO-8859-1"; format="flowed"
Content-Transfer-Encoding: 7bit
Cc: "draft-ietf-simple-chat.all@tools.ietf.org" <draft-ietf-simple-chat.all@tools.ietf.org>, IESG <iesg@ietf.org>, "apps-discuss@ietf.org" <apps-discuss@ietf.org>
Subject: Re: [apps-discuss] APPSDIR review of draft-ietf-simple-chat-13
X-BeenThere: apps-discuss@ietf.org
X-Mailman-Version: 2.1.12
Precedence: list
List-Id: General discussion of application-layer protocols <apps-discuss.ietf.org>
List-Unsubscribe: <https://www.ietf.org/mailman/options/apps-discuss>, <mailto:apps-discuss-request@ietf.org?subject=unsubscribe>
List-Archive: <http://www.ietf.org/mail-archive/web/apps-discuss>
List-Post: <mailto:apps-discuss@ietf.org>
List-Help: <mailto:apps-discuss-request@ietf.org?subject=help>
List-Subscribe: <https://www.ietf.org/mailman/listinfo/apps-discuss>, <mailto:apps-discuss-request@ietf.org?subject=subscribe>
X-List-Received-Date: Wed, 08 Feb 2012 18:17:15 -0000
Hi Miguel, On 07/02/2012 14:12, Miguel A. Garcia wrote: > Hi Enrico: > > Thanks for your review. > > See inline comments. > > On 06/02/2012 10:50, Enrico Marocco wrote: >> Document: draft-ietf-simple-chat-13 >> Title: Multi-party Chat Using the Message Session Relay Protocol (MSRP) >> Reviewers: Alexey Melnicov and Enrico Marocco >> Review Date: 2012-02-06 >> IETF Last Call Date: 2012-02-06 >> >> >> Summary: This draft is almost ready for publication as a Proposed >> Standard, but has a major issue to be taken into consideration and a few >> minor issues to be fixed. >> >> >> Major Issue >> >> The document doesn't describe allowed characters in Nicks and any >> normalization that needs to be applied. > > I think the document clearly describes the allowed characters in > nicknames by making a reference to RFC 4975. In Section 7.1, the > syntax of the Use-Nickname header is written as: > > Use-Nickname = "Use-Nickname:" SP nickname > nickname = quoted-string > ; quoted-string defined in RFC 4975 > > According to RFC 4975: > > quoted-string = DQUOTE *(qdtext / qd-esc) DQUOTE > qdtext = SP / HTAB / %x21 / %x23-5B / %x5D-7E > / UTF8-NONASCII > qd-esc = (BACKSLASH BACKSLASH) / (BACKSLASH DQUOTE) > BACKSLASH = "\" > UPALPHA = %x41-5A > ALPHANUM = ALPHA / DIGIT > > > So, I think it is described. If ABNF is saying that only UTF-8 is allowed, then the current text is Ok. > > With respect to normalization, I am not sure if a nickname needs to be > normalized and now this normalization should take place. Let me know > the text that you are missing and I will add it to the draft. Have a look at ResourcePrep: http://datatracker.ietf.org/doc/draft-ietf-xmpp-6122bis/ If you can't wait for Precis to finish, you can also use an older version (based on StringPrep): http://www.rfc-editor.org/rfc/rfc6122.txt
- [apps-discuss] APPSDIR review of draft-ietf-simpl… Enrico Marocco
- Re: [apps-discuss] APPSDIR review of draft-ietf-s… Ben Campbell
- Re: [apps-discuss] APPSDIR review of draft-ietf-s… Miguel A. Garcia
- Re: [apps-discuss] APPSDIR review of draft-ietf-s… Ben Campbell
- Re: [apps-discuss] APPSDIR review of draft-ietf-s… Miguel A. Garcia
- Re: [apps-discuss] APPSDIR review of draft-ietf-s… Alexey Melnikov
- Re: [apps-discuss] APPSDIR review of draft-ietf-s… Miguel A. Garcia
- Re: [apps-discuss] APPSDIR review of draft-ietf-s… Alexey Melnikov