Re: [pcp] #76 (proxy): Dave Thaler's comments on draft-ietf-pcp-proxy-06

<mohamed.boucadair@orange.com> Wed, 29 April 2015 08:02 UTC

Return-Path: <mohamed.boucadair@orange.com>
X-Original-To: pcp@ietfa.amsl.com
Delivered-To: pcp@ietfa.amsl.com
Received: from localhost (ietfa.amsl.com [127.0.0.1]) by ietfa.amsl.com (Postfix) with ESMTP id 61C911B2BF0 for <pcp@ietfa.amsl.com>; Wed, 29 Apr 2015 01:02:11 -0700 (PDT)
X-Virus-Scanned: amavisd-new at amsl.com
X-Spam-Flag: NO
X-Spam-Score: -2.599
X-Spam-Level:
X-Spam-Status: No, score=-2.599 tagged_above=-999 required=5 tests=[BAYES_00=-1.9, FREEMAIL_FROM=0.001, RCVD_IN_DNSWL_LOW=-0.7, SPF_PASS=-0.001, UNPARSEABLE_RELAY=0.001] 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 exjz2TgLf_e1 for <pcp@ietfa.amsl.com>; Wed, 29 Apr 2015 01:02:09 -0700 (PDT)
Received: from relais-inet.francetelecom.com (relais-ias92.francetelecom.com [193.251.215.92]) (using TLSv1.2 with cipher AECDH-AES256-SHA (256/256 bits)) (No client certificate requested) by ietfa.amsl.com (Postfix) with ESMTPS id 995C01B2C02 for <pcp@ietf.org>; Wed, 29 Apr 2015 01:02:09 -0700 (PDT)
Received: from omfedm07.si.francetelecom.fr (unknown [xx.xx.xx.3]) by omfedm13.si.francetelecom.fr (ESMTP service) with ESMTP id EB17E3240CF; Wed, 29 Apr 2015 10:02:07 +0200 (CEST)
Received: from Exchangemail-eme2.itn.ftgroup (unknown [10.114.31.18]) by omfedm07.si.francetelecom.fr (ESMTP service) with ESMTP id C7A464C015; Wed, 29 Apr 2015 10:02:07 +0200 (CEST)
Received: from OPEXCLILMA3.corporate.adroot.infra.ftgroup ([fe80::60a9:abc3:86e6:2541]) by OPEXCLILM34.corporate.adroot.infra.ftgroup ([fe80::cba:56d0:a732:ef5a%19]) with mapi id 14.03.0235.001; Wed, 29 Apr 2015 10:02:07 +0200
From: mohamed.boucadair@orange.com
To: pcp issue tracker <trac@tools.ietf.org>, "draft-ietf-pcp-proxy@tools.ietf.org" <draft-ietf-pcp-proxy@tools.ietf.org>, "dthaler@microsoft.com" <dthaler@microsoft.com>
Thread-Topic: [pcp] #76 (proxy): Dave Thaler's comments on draft-ietf-pcp-proxy-06
Thread-Index: AQHQgftdtOkWX1YXwUOT9oB40oK5hZ1joN5A
Date: Wed, 29 Apr 2015 08:02:06 +0000
Message-ID: <787AE7BB302AE849A7480A190F8B93300530C421@OPEXCLILMA3.corporate.adroot.infra.ftgroup>
References: <059.537358a39f79e82289de47eacc67bfba@tools.ietf.org>
In-Reply-To: <059.537358a39f79e82289de47eacc67bfba@tools.ietf.org>
Accept-Language: fr-FR, en-US
Content-Language: fr-FR
X-MS-Has-Attach:
X-MS-TNEF-Correlator:
x-originating-ip: [10.168.234.1]
Content-Type: text/plain; charset="iso-8859-1"
Content-Transfer-Encoding: quoted-printable
MIME-Version: 1.0
X-PMX-Version: 6.2.1.2478543, Antispam-Engine: 2.7.2.2107409, Antispam-Data: 2015.4.29.44529
Archived-At: <http://mailarchive.ietf.org/arch/msg/pcp/BZXN8hHTe4dYla5d5dIrKH8Iofw>
Cc: "pcp@ietf.org" <pcp@ietf.org>
Subject: Re: [pcp] #76 (proxy): Dave Thaler's comments on draft-ietf-pcp-proxy-06
X-BeenThere: pcp@ietf.org
X-Mailman-Version: 2.1.15
Precedence: list
List-Id: PCP wg discussion list <pcp.ietf.org>
List-Unsubscribe: <https://www.ietf.org/mailman/options/pcp>, <mailto:pcp-request@ietf.org?subject=unsubscribe>
List-Archive: <http://www.ietf.org/mail-archive/web/pcp/>
List-Post: <mailto:pcp@ietf.org>
List-Help: <mailto:pcp-request@ietf.org?subject=help>
List-Subscribe: <https://www.ietf.org/mailman/listinfo/pcp>, <mailto:pcp-request@ietf.org?subject=subscribe>
X-List-Received-Date: Wed, 29 Apr 2015 08:02:11 -0000

Hi Dave,

Thank you for the review. 

I integrated almost all of your comments except the one about Third Party ID Option because IMHO it does not address the same problem as zone_id. Anyway, zone_id is provided as an example, so we don't need to ne exhaustive here. 

FWIW, the diff is available here:           https:https://www.ietf.org/rfcdiff?url2=draft-ietf-pcp-proxy-07  

Cheers,
Med

> -----Message d'origine-----
> De : pcp [mailto:pcp-bounces@ietf.org] De la part de pcp issue tracker
> Envoyé : mardi 28 avril 2015 23:36
> À : draft-ietf-pcp-proxy@tools.ietf.org; dthaler@microsoft.com
> Cc : pcp@ietf.org
> Objet : [pcp] #76 (proxy): Dave Thaler's comments on draft-ietf-pcp-proxy-
> 06
> 
> #76: Dave Thaler's comments on draft-ietf-pcp-proxy-06
> 
>  Section 1.1:
>  > likely to be more than on NAT on the path between client machines and
> 
>  s/more than on/more than one/
> 
>  Section 1.2:
>  Is it worth noting that the pcp-anycast draft is an alternate solution to
>  this same use case?
> 
>  Section 3.2:
>  This section's wording is a bit confusing.   Section 1 explained the
>  reference
>  model of back-to-back client and server.  So termination would occur when
>  either
>  a) the request gets to a server that does not have the PCP proxy
>  functional element, or
>  b) it gets to a PCP proxy whose PCP client has no PCP servers reachable
>  (e.g. none configured and no default routers that respond to PCP) and
> thus
>  has nothing to forward to.
>  No?
> 
>  The two examples currently in 3.2 are merely ways a NAT gateway (but a
> PCP
>  proxy could be a firewall or something other than a NAT...) could choose
>  to enable or disable a PCP proxy functional element and hence
>  toggle between a and b above.   But the PCP speaker might not be a NAT
>  gateway, or it might not implement a PCP proxy at all, or it might
>  implement one but not be "configured to know it's the outermost" but
>  rather use rule b above.
>  So I think section 3.2 needs further elaboration as noted above.
> 
>  Section 3.6:
>  I think the reference to Zone ID [I-D.penno-pcp-zones] should probably
>  reference the Third Party ID Option [I-D.ietf-pcp-third-party-id-option].
> 
>  Section 5:
>  > Section 3.3 specifies the cases where a THIRD_PARTY option is inserted
>  > the PCP Proxy.  In those cases, means to prevent a malicious
> 
>  s/inserted/inserted by/
> 
>  > THIRD_PARTY option MUST NOT be enabled unless the network on which
> 
>  Grammar: s/option/options/
> 
>  >   dropped (or in the case of an unknown Option which is not mandatory-
>  >   to-process the Option be removed) if it is not compatible with
> 
>  s/Option be/Option SHOULD be/
> 
> --
> -------------------------------------+------------------------------------
> -
>  Reporter:  dthaler@microsoft.com    |      Owner:  draft-ietf-pcp-
>      Type:  defect                   |  proxy@tools.ietf.org
>  Priority:  minor                    |     Status:  new
> Component:  proxy                    |  Milestone:  milestone1
>  Severity:  Waiting for Shepherd     |    Version:  1.0
>   Writeup                            |   Keywords:
> -------------------------------------+------------------------------------
> -
> 
> Ticket URL: <http://tools.ietf.org/wg/pcp/trac/ticket/76>
> pcp <http://tools.ietf.org/pcp/>
> 
> _______________________________________________
> pcp mailing list
> pcp@ietf.org
> https://www.ietf.org/mailman/listinfo/pcp