[Gen-art] Gen-art LC review of draft-ietf-mmusic-latching-05
Elwyn Davies <elwynd@dial.pipex.com> Tue, 27 May 2014 16:35 UTC
Return-Path: <elwynd@dial.pipex.com>
X-Original-To: gen-art@ietfa.amsl.com
Delivered-To: gen-art@ietfa.amsl.com
Received: from localhost (ietfa.amsl.com [127.0.0.1]) by ietfa.amsl.com (Postfix) with ESMTP id 0B7731A04AA for <gen-art@ietfa.amsl.com>; Tue, 27 May 2014 09:35:27 -0700 (PDT)
X-Virus-Scanned: amavisd-new at amsl.com
X-Spam-Flag: NO
X-Spam-Score: -101.9
X-Spam-Level:
X-Spam-Status: No, score=-101.9 tagged_above=-999 required=5 tests=[BAYES_00=-1.9, USER_IN_WHITELIST=-100] 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 IYkFjPbWaiL7 for <gen-art@ietfa.amsl.com>; Tue, 27 May 2014 09:35:24 -0700 (PDT)
Received: from auth.a.painless.aa.net.uk (a.painless.aa.net.uk [IPv6:2001:8b0:0:30::51bb:1e33]) (using TLSv1 with cipher DHE-RSA-AES256-SHA (256/256 bits)) (No client certificate requested) by ietfa.amsl.com (Postfix) with ESMTPS id CEEB21A01CB for <gen-art@ietf.org>; Tue, 27 May 2014 09:35:19 -0700 (PDT)
Received: from mightyatom.folly.org.uk ([81.187.254.250]) by a.painless.aa.net.uk with esmtpsa (TLSv1:AES256-SHA:256) (Exim 4.77) (envelope-from <elwynd@dial.pipex.com>) id 1WpKLB-0008Ii-UG; Tue, 27 May 2014 17:35:09 +0100
From: Elwyn Davies <elwynd@dial.pipex.com>
To: General Area Review Team <gen-art@ietf.org>
Content-Type: text/plain
Date: Tue, 27 May 2014 17:35:03 +0100
Message-Id: <1401208503.29419.2830.camel@mightyatom>
Mime-Version: 1.0
X-Mailer: Evolution 2.26.3
Content-Transfer-Encoding: 7bit
Archived-At: http://mailarchive.ietf.org/arch/msg/gen-art/goTgL2hqMWUbFtACmeO3iyhtESs
Cc: draft-ietf-mmusic-latching.all@tools.ietf.org, Barry Leiba <barryleiba@computer.org>
Subject: [Gen-art] Gen-art LC review of draft-ietf-mmusic-latching-05
X-BeenThere: gen-art@ietf.org
X-Mailman-Version: 2.1.15
Precedence: list
List-Id: "GEN-ART: General Area Review Team" <gen-art.ietf.org>
List-Unsubscribe: <https://www.ietf.org/mailman/options/gen-art>, <mailto:gen-art-request@ietf.org?subject=unsubscribe>
List-Archive: <http://www.ietf.org/mail-archive/web/gen-art/>
List-Post: <mailto:gen-art@ietf.org>
List-Help: <mailto:gen-art-request@ietf.org?subject=help>
List-Subscribe: <https://www.ietf.org/mailman/listinfo/gen-art>, <mailto:gen-art-request@ietf.org?subject=subscribe>
X-List-Received-Date: Tue, 27 May 2014 16:35:27 -0000
I am the assigned Gen-ART reviewer for this draft. For background on Gen-ART, please see the FAQ at <http://wiki.tools.ietf.org/area/gen/trac/wiki/GenArtfaq>. Please resolve these comments along with any other Last Call comments you may receive. Document: draft-ietf-mmusic-latching-05.txt Reviewer: Elwyn Davies Review Date: 27 May 2014 IETF LC End Date: 28 May 2014 IESG Telechat date: 29 May 2014 Summary: Ready with nits. Generally a well argued document. In the light of the comment that the IETF advises the use of ICE or STUN rather than HNT, I wondered if it might be helpful to explain how these mechanisms mitigate or resolve the security issues in s5. Major issues: None Minor issues: s5: Section 4 talks about problems with XMPP but the security concerns in s5 are all discussed in the context of SIP/SBC. I think some words about the corresponding security issues in XMPP (or just a statement that all - or a subset - of these apply to XMPP) ought to be added. s8: <Heresy Starts> Barry Leiba in his comments in the tracker suggests that the references would be usefully split into Normative and Informative subsets. Given the number of references, splitting them up seems like a good idea. I am going to suggest something highly heretical: Split them up but call them "Key References" and "Additional References". "Normative" has become such a loaded word in the standards community that, despite its underlying English meaning, it is probably better to confine its usage to Standards Track documents. I feel that we usefully adopt this alternative classification for non-standards track documents as a general technique to avoid the ongoing discussions about split refetrence sections. <Heresy Ends> Nits/editorial comments: General: Would probably be useful to explain the "address:port" terminology; Also both the terms "couple" and "set" are used for the tuple - better to stick with one and use "address:port sets/couples" instead of "address:ports". General: s/e.g./e.g.,/g s1: Need to expand SDP s3, last sentence: OLD: the SBC may decide not to send media to that customer UA until a SIP 200 response for policy reasons, to prevent toll-fraud. NEW: the SBC may decide, for policy reasons, not to send media to that customer UA until a SIP 200 response has been received, [e.g., ???] to prevent toll-fraud. s4, para 1: s/ address:port set that once packets cross the NAT, will be mapped/address:port set that, once packets cross the NAT, will be mapped/ s4, Figure 2: The figure doesn't reflect the address mapping done in the NATs. the clients Alice and Bob are shown with public (documentation) addresses whereas they should presumably have private addresses that are mapped to these public addresses by the NAT. s4, Figure 3: The previous comment doesn't apply to this figure which shows the NAT mapping. Arguably it would be nice to use private address space addresses on the private side of the NAT, but I notice we never managed to allocate a specific private address for documentation - I suppose since they are private it doesn't matter! s4, Figure 3, title: Cut and paste error... this figure is about XMPP not SBC! s5, para 2: s/In all/All/
- [Gen-art] Gen-art LC review of draft-ietf-mmusic-… Elwyn Davies
- Re: [Gen-art] Gen-art LC review of draft-ietf-mmu… Barry Leiba
- Re: [Gen-art] Gen-art LC review of draft-ietf-mmu… Jari Arkko
- Re: [Gen-art] Gen-art LC review of draft-ietf-mmu… Emil Ivov
- Re: [Gen-art] Gen-art LC review of draft-ietf-mmu… Barry Leiba
- Re: [Gen-art] Gen-art LC review of draft-ietf-mmu… Emil Ivov
- Re: [Gen-art] Gen-art LC review of draft-ietf-mmu… Elwyn Davies
- Re: [Gen-art] Gen-art LC review of draft-ietf-mmu… Emil Ivov
- Re: [Gen-art] Gen-art LC review of draft-ietf-mmu… Elwyn Davies
- Re: [Gen-art] Gen-art LC review of draft-ietf-mmu… Emil Ivov