Re: [Gen-art] Genart last call review of draft-ietf-sipcore-originating-cdiv-parameter-05
Alissa Cooper <alissa@cooperw.in> Wed, 05 December 2018 21:45 UTC
Return-Path: <alissa@cooperw.in>
X-Original-To: gen-art@ietfa.amsl.com
Delivered-To: gen-art@ietfa.amsl.com
Received: from localhost (localhost [127.0.0.1]) by ietfa.amsl.com (Postfix) with ESMTP id E70C4130DFF; Wed, 5 Dec 2018 13:45:28 -0800 (PST)
X-Virus-Scanned: amavisd-new at amsl.com
X-Spam-Flag: NO
X-Spam-Score: -2.701
X-Spam-Level:
X-Spam-Status: No, score=-2.701 tagged_above=-999 required=5 tests=[BAYES_00=-1.9, DKIM_SIGNED=0.1, DKIM_VALID=-0.1, DKIM_VALID_AU=-0.1, RCVD_IN_DNSWL_LOW=-0.7, SPF_PASS=-0.001] autolearn=ham autolearn_force=no
Authentication-Results: ietfa.amsl.com (amavisd-new); dkim=pass (2048-bit key) header.d=cooperw.in header.b=g5k59ogA; dkim=pass (2048-bit key) header.d=messagingengine.com header.b=xOYg+p48
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 1iuxZ_P0OFV8; Wed, 5 Dec 2018 13:45:26 -0800 (PST)
Received: from out4-smtp.messagingengine.com (out4-smtp.messagingengine.com [66.111.4.28]) (using TLSv1.2 with cipher AECDH-AES256-SHA (256/256 bits)) (No client certificate requested) by ietfa.amsl.com (Postfix) with ESMTPS id 2B5B3130E03; Wed, 5 Dec 2018 13:45:26 -0800 (PST)
Received: from compute7.internal (compute7.nyi.internal [10.202.2.47]) by mailout.nyi.internal (Postfix) with ESMTP id 1155A23F9B; Wed, 5 Dec 2018 16:45:25 -0500 (EST)
Received: from mailfrontend2 ([10.202.2.163]) by compute7.internal (MEProxy); Wed, 05 Dec 2018 16:45:25 -0500
DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=cooperw.in; h= content-type:mime-version:subject:from:in-reply-to:date:cc :content-transfer-encoding:message-id:references:to; s=fm1; bh=A KM2Ao4ZGp69Sv+6w16nOT5bwAN+p1QmHdGG6a9GZ6E=; b=g5k59ogAU1nntagLP UgwrZJe2kN0MqCu3GkXakjsZ+dC6An9YSu65hlfsVl5OlQoN2MXdO0mhgybkqMqr iy683oB9MSJzsL1X3pNWjkK0kBDUg5ddY393VZOfdxUWasVFzWo5mvcSqUVYWsDN 7ekqCwmzJABnWblR0i2nAcFAhn2kKx6Os2Kr3cNYmL3yUn3IGhWDHvW3R9vBnFWZ gv6rkbqkELixbm4T3IYUk1XU1BfLe/fGqMoJJDAw2nTMkFTnYWOB/hBLxHtrSK+E f6tNI+l+KGp06LOGBSXlEIWt+iOi5sf25gpSFPeXgECnzLsWNUy9tyFx6XVokHzv OpQZg==
DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d= messagingengine.com; h=cc:content-transfer-encoding:content-type :date:from:in-reply-to:message-id:mime-version:references :subject:to:x-me-proxy:x-me-proxy:x-me-sender:x-me-sender :x-sasl-enc; s=fm1; bh=AKM2Ao4ZGp69Sv+6w16nOT5bwAN+p1QmHdGG6a9GZ 6E=; b=xOYg+p48KANCQ7WWAk6pk7PD9WLrvfE19YTh/9MszcOfqpV2kfu696Y0A 4T0LXWVvVpn8TEzGv3OsjXACRl5KGa0kk969alpyCO058Xvoibx7TdGOsyXLrs5i BDKsV67eOPkqSUHZUeQC/jcTHLQwxaBGJ16IodC4kV0PwquiniuM1tU+Vq4j9wIx ik40ifuqM80ACbcKciLjRMsITfgFc1x1o4aZ76IxnSw1ho1BZ/hPljkttW0ziI8j KRaD+V10Dfj6KbiNoQoBpz2U157njYzCzymyZla7/DOlV+4bVb15rc5/Qx5ROsCF QEof/0jmE1KhXr4gVJOoDDWAM+pqw==
X-ME-Sender: <xms:9EYIXF2E75W9l2TWiNidoU2yA2pcuaV4W6IGclv_n9WZWYy1vLHExQ>
X-ME-Proxy: <xmx:9EYIXH4hh2D6-j5W1KrumpPit03bZy7PP7lF4fAd5l59fqaMzl2L4g> <xmx:9EYIXDxd9cDRnMP6WS10AP09mBSTmSjKuT_DxHsbYVAuGFbB6MnfBA> <xmx:9EYIXGwtnzir-wu8bOFOvipBIrA1sZZSgtyFLraNICc1lhONLrPdrg> <xmx:9EYIXDyJxivp02XlvZRQ0EcUB8KUo-Z9EnuiVpczA8gzsm1_gbQs_w> <xmx:9EYIXIYTOsNJJXgOe15XEBL-LM3bO6AlHaYah1BJtgSPvstRzpbfZQ> <xmx:9UYIXPyRk4mRSSDNmQidJau31XA8H3uEldruDi3fFEKCU08fWphSYQ>
Received: from rtp-alcoop-nitro5.cisco.com (unknown [173.38.117.89]) by mail.messagingengine.com (Postfix) with ESMTPA id 28552102E4; Wed, 5 Dec 2018 16:45:24 -0500 (EST)
Content-Type: text/plain; charset="utf-8"
Mime-Version: 1.0 (Mac OS X Mail 11.5 \(3445.9.1\))
From: Alissa Cooper <alissa@cooperw.in>
In-Reply-To: <9722_1541485853_5BE1351D_9722_415_1_8B970F90C584EA4E97D5BAAC9172DBB84A256528@OPEXCLILMA4.corporate.adroot.infra.ftgroup>
Date: Wed, 05 Dec 2018 16:45:21 -0500
Cc: Ben Campbell <ben@nostrum.com>, "gen-art@ietf.org" <gen-art@ietf.org>, "sipcore@ietf.org" <sipcore@ietf.org>, "draft-ietf-sipcore-originating-cdiv-parameter.all@ietf.org" <draft-ietf-sipcore-originating-cdiv-parameter.all@ietf.org>
Content-Transfer-Encoding: quoted-printable
Message-Id: <A9F4D098-FFC4-4C81-A267-DE73F7F43A65@cooperw.in>
References: <154084621265.4963.11856647404649644525@ietfa.amsl.com> <26344_1541435406_5BE0700E_26344_158_14_8B970F90C584EA4E97D5BAAC9172DBB84A2558EC@OPEXCLILMA4.corporate.adroot.infra.ftgroup> <CAMMTW_Ly35VdhH3bfHvYdFVijLTuS1hYJTyoFMzT99T=i7YhCg@mail.gmail.com> <19441_1541437458_5BE07812_19441_188_1_8B970F90C584EA4E97D5BAAC9172DBB84A255B4D@OPEXCLILMA4.corporate.adroot.infra.ftgroup> <CAMMTW_LF61b0BrTtigK0A8nAsMZCt6=W_Y=8CoDM2z5_fR_4-g@mail.gmail.com> <21449_1541440262_5BE08306_21449_405_1_8B970F90C584EA4E97D5BAAC9172DBB84A255C43@OPEXCLILMA4.corporate.adroot.infra.ftgroup> <CAMMTW_K9Vhib_Dksfv0QCVE4-Gq0Auj4goYF6Vt2wski-feDLw@mail.gmail.com> <EC9E0646-95B7-448C-B65C-968D5407D8E7@nostrum.com> <9722_1541485853_5BE1351D_9722_415_1_8B970F90C584EA4E97D5BAAC9172DBB84A256528@OPEXCLILMA4.corporate.adroot.infra.ftgroup>
To: marianne.mohali@orange.com
X-Mailer: Apple Mail (2.3445.9.1)
Archived-At: <https://mailarchive.ietf.org/arch/msg/gen-art/VsS8dqxYjp0mON1u9ycvoZAXjE8>
Subject: Re: [Gen-art] Genart last call review of draft-ietf-sipcore-originating-cdiv-parameter-05
X-BeenThere: gen-art@ietf.org
X-Mailman-Version: 2.1.29
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: <https://mailarchive.ietf.org/arch/browse/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: Wed, 05 Dec 2018 21:45:29 -0000
Vijay, thank you for your review. Marianne, thank you for your responses. I entered a No Objection ballot. Alissa > On Nov 6, 2018, at 1:30 AM, marianne.mohali@orange.com wrote: > > Ok no problem, I keep it in mind for a further update of the I-D. > > Marianne > > -----Message d'origine----- > De : Ben Campbell [mailto:ben@nostrum.com] > Envoyé : mardi 6 novembre 2018 02:18 > À : MOHALI Marianne TGI/OLN > Cc : gen-art@ietf.org; sipcore@ietf.org; IETF list; Vijay Gurbani; draft-ietf-sipcore-originating-cdiv-parameter.all@ietf.org; A. Jean Mahoney > Objet : Re: Genart last call review of draft-ietf-sipcore-originating-cdiv-parameter-05 > > Editorial comment: > > Since “originating after CDIV” is effectively used as a compound adjective, it would be better to hyphenate it, as in “originating-after-CDIV session”. That might also make it less confusing to people unfamiliar with the terminology. > > (Such a change can wait to be handled along with any IESG review comments.) > > Thanks! > > Ben. > >> On Nov 6, 2018, at 12:57 AM, Vijay Gurbani <vijay.gurbani@gmail.com> wrote: >> >> Dear Marianne: OK, if the context of "originating after CDIV" is well understood by the folks working in this area, then I am fine with leaving it as is. >> Thanks. >> - vijay >> >> On Mon, Nov 5, 2018 at 11:51 AM <marianne.mohali@orange.com> wrote: >> Dear Vijay, >> >> >> >> Actually, the « originating » is not qualifying something by itself in this sentence, it has to be understood as a global wording for the new defined session case which is "originating after CDIV" which is different for an “originating call leg”. >> >> >> >> If you don’t mind, I would prefer to keep this wording as it is because it is used although the I-D and quoted in the Introduction section in the following sentence: >> >> "The sessioncase-param parameter of the P-Served-User header field is extended with the "orig-cdiv" parameter for this "originating after CDIV" session case." >> >> >> >> Marianne >> >> >> >> De : Vijay Gurbani [mailto:vijay.gurbani@gmail.com] Envoyé : lundi 5 >> novembre 2018 18:14 À : MOHALI Marianne TGI/OLN Cc : gen-art@ietf.org; >> sipcore@ietf.org; ietf@ietf.org; >> draft-ietf-sipcore-originating-cdiv-parameter.all@ietf.org; Jean >> Mahoney; ben@nostrum.com Objet : Re: Genart last call review of >> draft-ietf-sipcore-originating-cdiv-parameter-05 >> >> >> >> Dear Marianne: Thank you, again, for attending to my comment. >> >> >> >> Note that you still have a dangling verb "originating" in the sentence. The verb is not qualifying anything: >> >> >> >> For this use case, this document creates a new parameter ("orig-cdiv") for >> the originating after CDIV session case to be embedded in the P-Served-User >> header field. >> >> >> >> In my email, I had suggested adding "call leg" after the "originating" above. Otherwise, the sentence above is incomplete ... "originating" what? >> >> >> >> Thanks. >> >> >> >> >> >> On Mon, Nov 5, 2018 at 11:04 AM <marianne.mohali@orange.com> wrote: >> >> Thanks Vijay for your last feedback. I’m fine with your proposal and have updated the I-D accordingly (v-07): >> https://datatracker.ietf.org/doc/draft-ietf-sipcore-originating-cdiv-p >> arameter/ >> >> BR, >> Marianne >> >> De : Vijay Gurbani [mailto:vijay.gurbani@gmail.com] Envoyé : lundi 5 >> novembre 2018 17:45 À : MOHALI Marianne TGI/OLN Cc : gen-art@ietf.org; >> sipcore@ietf.org; ietf@ietf.org; >> draft-ietf-sipcore-originating-cdiv-parameter.all@ietf.org >> Objet : Re: Genart last call review of >> draft-ietf-sipcore-originating-cdiv-parameter-05 >> >> Dear Marianne: Thank you for attending to my comments. >> >> I am fine with the text you added for S1.3. >> >> Regarding "secase" and "regstate" being existing parameters, ok. However, since the I-D is defining the "orig-cdiv" parameter, I still think it makes sense to mention this before S4. You already have the text at the end of S1.3 (the current sentence appears ambiguous). Let me suggest an edit: >> >> OLD: >> For this use case, this document creates a new parameter for the >> originating after CDIV session case to be embedded in the P-Served- >> User header field. >> >> NEW: >> For this use case, this document creates a new parameter ("orig-cdiv") for the >> originating call leg to be embedded in the P-Served-User header field. >> Thanks. >> >> On Mon, Nov 5, 2018 at 10:30 AM <marianne.mohali@orange.com> wrote: >> Hi all, >> >> Thanks Vijay for the GenArt review. >> I've just submitted a v-06 to address your comments and here is my feedbacks: >> https://datatracker.ietf.org/doc/draft-ietf-sipcore-originating-cdiv-p >> arameter/ >> >>> Minor: >>> >>> - S1.3: I am not sure I follow the logic in the problem statement. >>> Who is the "diverting" user? The user to who the call was destined? >>> If so, best to say that explicitly. (To be sure, I looked into >>> rfc5502 as well, and it does not define "diverting" user either.) A >>> bit below (in S4), you use the term "served" user to refer to the >>> diverting user. All in all, the terminology here could be refined. >>> I suspect that the "originating" user is the callee. >>> >>> Concretely, I think that the first paragraph of S1.3 should be >>> re-written, perhaps with a figure (?) to explain the call flow, or >>> at least some context using Alice, Bob and Carol as the example in >>> S7.1 does (I suspect that Carol is the "diverting" user here). >> >> [MM] Indeed, I can see that for people not very aware of IETF and 3GPP vocabulary for call diversion service, it can be confusing. I prefer not to add a call flow in the problem statement section but I did some updates in the wording and inserted the Alice, Bob and Carol users for a better understanding. >> >>> Nits, typos: >>> >>> - S4, step 3: s/user an INVITE that/user as an INVITE that/ Also, >>> the "secase" and "regstate" parameters are what you are standardizing >>> this I-D, as such you mention this before S4 so the reader knows that >>> these are the new parameters. Same for "orig-cdiv" parameter. >> >> [MM] Nits is corrected. About your comment, actually, this I-D is only standardizing "orig-cdiv" parameter. This is the reason why "sescase" and "regstate" appear, as part of a normal session establishment and before any call diversion while the new parameter can appear only when this event occurs (as added by this I-D).. I hope it's clearer for you. >> >> I hope it's ok. >> >> Best regards, >> Marianne >> >> -----Message d'origine----- >> De : Vijay Gurbani [mailto:vijay.gurbani@gmail.com] Envoyé : lundi 29 >> octobre 2018 21:50 À : gen-art@ietf.org Cc : sipcore@ietf.org; >> ietf@ietf.org; >> draft-ietf-sipcore-originating-cdiv-parameter.all@ietf.org >> Objet : Genart last call review of >> draft-ietf-sipcore-originating-cdiv-parameter-05 >> >> Reviewer: Vijay Gurbani >> Review result: Almost Ready >> >> I am the assigned Gen-ART reviewer for this draft. The General Area >> Review Team (Gen-ART) reviews all IETF documents being processed by >> the IESG for the IETF Chair. Please treat these comments just like >> any other last call comments. >> >> For more information, please see the FAQ at >> >> <https://trac.ietf.org/trac/gen/wiki/GenArtfaq>. >> >> Document: draft-ietf-sipcore-originating-cdiv-parameter-?? >> Reviewer: Vijay K. Gurbani >> Review Date: 2018-10-29 >> IETF LC End Date: 2018-10-26 >> IESG Telechat date: Not scheduled for a telechat >> >> Summary: This draft is on the right track but has open issues, described in the review. >> >> Major issues: 0 >> >> Minor issues: 1 >> >> Nits/editorial comments: 1 >> >> Minor: >> >> - S1.3: I am not sure I follow the logic in the problem statement. >> Who is the "diverting" user? The user to who the call was destined? >> If so, best to say that explicitly. (To be sure, I looked into >> rfc5502 as well, and it does not define "diverting" user either.) A >> bit below (in S4), you use the term "served" user to refer to the >> diverting user. All in all, the terminology here could be refined. >> I suspect that the "originating" user is the callee. >> >> Concretely, I think that the first paragraph of S1.3 should be >> re-written, perhaps with a figure (?) to explain the call flow, or at >> least some context using Alice, Bob and Carol as the example in S7.1 >> does (I suspect that Carol is the "diverting" user here). >> >> Nits, typos: >> >> - S4, step 3: s/user an INVITE that/user as an INVITE that/ Also, the >> "secase" and "regstate" parameters are what you are standardizing >> this I-D, as such you mention this before S4 so the reader knows that >> these are the new parameters. Same for "orig-cdiv" parameter. >> >> >> ______________________________________________________________________ >> ___________________________________________________ >> >> Ce message et ses pieces jointes peuvent contenir des informations >> confidentielles ou privilegiees et ne doivent donc pas etre diffuses, >> exploites ou copies sans autorisation. Si vous avez recu ce message >> par erreur, veuillez le signaler a l'expediteur et le detruire ainsi que les pieces jointes. Les messages electroniques etant susceptibles d'alteration, Orange decline toute responsabilite si ce message a ete altere, deforme ou falsifie. Merci. >> >> This message and its attachments may contain confidential or >> privileged information that may be protected by law; they should not be distributed, used or copied without authorisation. >> If you have received this email in error, please notify the sender and delete this message and its attachments. >> As emails may be altered, Orange is not liable for messages that have been modified, changed or falsified. >> Thank you. >> >> ______________________________________________________________________ >> ___________________________________________________ >> >> Ce message et ses pieces jointes peuvent contenir des informations >> confidentielles ou privilegiees et ne doivent donc pas etre diffuses, >> exploites ou copies sans autorisation. Si vous avez recu ce message >> par erreur, veuillez le signaler a l'expediteur et le detruire ainsi que les pieces jointes. Les messages electroniques etant susceptibles d'alteration, Orange decline toute responsabilite si ce message a ete altere, deforme ou falsifie. Merci. >> >> This message and its attachments may contain confidential or >> privileged information that may be protected by law; they should not be distributed, used or copied without authorisation. >> If you have received this email in error, please notify the sender and delete this message and its attachments. >> As emails may be altered, Orange is not liable for messages that have been modified, changed or falsified. >> Thank you. >> >> ______________________________________________________________________ >> ___________________________________________________ >> >> Ce message et ses pieces jointes peuvent contenir des informations >> confidentielles ou privilegiees et ne doivent donc pas etre diffuses, >> exploites ou copies sans autorisation. Si vous avez recu ce message >> par erreur, veuillez le signaler a l'expediteur et le detruire ainsi que les pieces jointes. Les messages electroniques etant susceptibles d'alteration, Orange decline toute responsabilite si ce message a ete altere, deforme ou falsifie. Merci. >> >> This message and its attachments may contain confidential or >> privileged information that may be protected by law; they should not be distributed, used or copied without authorisation. >> If you have received this email in error, please notify the sender and delete this message and its attachments. >> As emails may be altered, Orange is not liable for messages that have been modified, changed or falsified. >> Thank you. >> > > > _________________________________________________________________________________________________________________________ > > Ce message et ses pieces jointes peuvent contenir des informations confidentielles ou privilegiees et ne doivent donc > pas etre diffuses, exploites ou copies sans autorisation. Si vous avez recu ce message par erreur, veuillez le signaler > a l'expediteur et le detruire ainsi que les pieces jointes. Les messages electroniques etant susceptibles d'alteration, > Orange decline toute responsabilite si ce message a ete altere, deforme ou falsifie. Merci. > > This message and its attachments may contain confidential or privileged information that may be protected by law; > they should not be distributed, used or copied without authorisation. > If you have received this email in error, please notify the sender and delete this message and its attachments. > As emails may be altered, Orange is not liable for messages that have been modified, changed or falsified. > Thank you. > > _______________________________________________ > Gen-art mailing list > Gen-art@ietf.org > https://www.ietf.org/mailman/listinfo/gen-art
- [Gen-art] Genart last call review of draft-ietf-s… Vijay Gurbani
- Re: [Gen-art] Genart last call review of draft-ie… marianne.mohali
- Re: [Gen-art] Genart last call review of draft-ie… Vijay Gurbani
- Re: [Gen-art] Genart last call review of draft-ie… marianne.mohali
- Re: [Gen-art] Genart last call review of draft-ie… Vijay Gurbani
- Re: [Gen-art] Genart last call review of draft-ie… marianne.mohali
- Re: [Gen-art] Genart last call review of draft-ie… Vijay Gurbani
- Re: [Gen-art] Genart last call review of draft-ie… Ben Campbell
- Re: [Gen-art] Genart last call review of draft-ie… marianne.mohali
- Re: [Gen-art] Genart last call review of draft-ie… Alissa Cooper