Re: [stir] Review of draft-ietf-stir-rph-emergency-services

Chris Wendt <chris-ietf@chriswendt.net> Tue, 25 August 2020 15:15 UTC

Return-Path: <chris-ietf@chriswendt.net>
X-Original-To: stir@ietfa.amsl.com
Delivered-To: stir@ietfa.amsl.com
Received: from localhost (localhost [127.0.0.1]) by ietfa.amsl.com (Postfix) with ESMTP id 957A73A0E27 for <stir@ietfa.amsl.com>; Tue, 25 Aug 2020 08:15:03 -0700 (PDT)
X-Virus-Scanned: amavisd-new at amsl.com
X-Spam-Flag: NO
X-Spam-Score: 0.105
X-Spam-Level:
X-Spam-Status: No, score=0.105 tagged_above=-999 required=5 tests=[DKIM_SIGNED=0.1, DKIM_VALID=-0.1, HTML_FONT_LOW_CONTRAST=0.001, HTML_MESSAGE=0.001, HTTPS_HTTP_MISMATCH=0.1, SPF_HELO_NONE=0.001, SPF_NONE=0.001, URIBL_BLOCKED=0.001] autolearn=no autolearn_force=no
Authentication-Results: ietfa.amsl.com (amavisd-new); dkim=pass (2048-bit key) header.d=chriswendt-net.20150623.gappssmtp.com
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 v6AzbLqgF7O8 for <stir@ietfa.amsl.com>; Tue, 25 Aug 2020 08:15:00 -0700 (PDT)
Received: from mail-qt1-x835.google.com (mail-qt1-x835.google.com [IPv6:2607:f8b0:4864:20::835]) (using TLSv1.2 with cipher ECDHE-RSA-AES128-GCM-SHA256 (128/128 bits)) (No client certificate requested) by ietfa.amsl.com (Postfix) with ESMTPS id 5C68A3A0E19 for <stir@ietf.org>; Tue, 25 Aug 2020 08:15:00 -0700 (PDT)
Received: by mail-qt1-x835.google.com with SMTP id i2so5962236qtb.8 for <stir@ietf.org>; Tue, 25 Aug 2020 08:15:00 -0700 (PDT)
DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=chriswendt-net.20150623.gappssmtp.com; s=20150623; h=from:message-id:mime-version:subject:date:in-reply-to:cc:to :references; bh=VZdeUuHC5kB8pUlfeHQHklORGaz6fdrdAgr+isVg5R8=; b=j3TUwhjsPlYtIXxHB0PE7ZmJ7y0aVhcTsi4WvN5itEhLXIzxgLj70q5qZuSgJLZbZC +MWMlIZP1jc/2s+sMohnA/FZKhd3OH/qG8lGOoeOG6dgyBh7y2uI34Nj3H9vlpkp1b7A t+3KLDV549mEGM7MF608tBDtkJ3xcW+HPkIOjryoywP2o6c4OXQOiHtwJXBdEswhTyyW KWJt5YLT5E6Z8XNYj9vfrMFeZyJ8XxV01pKfBkzYUWCLDvSSubXfqa2T5t8iZgUpG/Im bmWvUTTneVrqu93pIBeHExv103xHSttzqH132uvBQTh5jJTT3Sp9SZr17pShtbt7BSmx ApEw==
X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20161025; h=x-gm-message-state:from:message-id:mime-version:subject:date :in-reply-to:cc:to:references; bh=VZdeUuHC5kB8pUlfeHQHklORGaz6fdrdAgr+isVg5R8=; b=eK2HsBksOJXl+aQ+N+U6UNUs1IWgZ1Ri3EeDXg7qhxTXunuOTCj0043jf18Sljv/1d RDwOTa/JRdIHNQww3Lv2iE7ckWW5aQXUmSLNqXj8VKBrI0FLqSbvb+MrVAqnSJwoR05W uQGFNtIHwrPsktSVJynEwLiOFdwLzGWAiMQWA4eNDUUchrZUQ3M0t14JHr0wRaoVWWgs GUhPLhLmOamwueu0eFZRc7VEj+UOYi1A7lKPf66hFQ4NNg+QY0gElKqrGdIYpNh6T7at FsA89jQAtsjkxMxOu4lOzgwyTcf3URX8qSZ27xo2Z0Idxuz+EmksD7RjnYpas5EjySe+ G02Q==
X-Gm-Message-State: AOAM531TVmOw/sg2ORb1lN3vApBCBZ0FKHkulrVZcq+4A+VazEHi8ZPN KgkUT5v+oMV1NI5v6/BvLg/UyQ==
X-Google-Smtp-Source: ABdhPJxES9JnMEtW7HhfR2sytTIj4fTgjQhnYtkzhtS3G5wXTB1BLgegufYLiU2Jy2EopC+3glmBkw==
X-Received: by 2002:ac8:5685:: with SMTP id h5mr10116094qta.378.1598368499354; Tue, 25 Aug 2020 08:14:59 -0700 (PDT)
Received: from ?IPv6:2607:fb90:6088:d6d3:c909:1b18:6467:10ec? ([2607:fb90:6088:d6d3:c909:1b18:6467:10ec]) by smtp.gmail.com with ESMTPSA id 95sm13776874qtc.29.2020.08.25.08.14.57 (version=TLS1_2 cipher=ECDHE-ECDSA-AES128-GCM-SHA256 bits=128/128); Tue, 25 Aug 2020 08:14:58 -0700 (PDT)
From: Chris Wendt <chris-ietf@chriswendt.net>
Message-Id: <14ACD074-FD66-4161-AC7A-ADB07127BE2D@chriswendt.net>
Content-Type: multipart/alternative; boundary="Apple-Mail=_A7F25D06-57D2-41F0-A6ED-AC3487DF6F4F"
Mime-Version: 1.0 (Mac OS X Mail 13.4 \(3608.120.23.2.1\))
Date: Tue, 25 Aug 2020 11:14:54 -0400
In-Reply-To: <BYAPR02MB5189D7055FCD08B0F926B9ADF35A0@BYAPR02MB5189.namprd02.prod.outlook.com>
Cc: Brian Rosen <br@brianrosen.net>, "draft-ietf-stir-rph-emergency-services@ietf.org" <draft-ietf-stir-rph-emergency-services@ietf.org>, "stir@ietf.org" <stir@ietf.org>
To: Jack Rickard <Jack.Rickard@metaswitch.com>
References: <BYAPR02MB51891E95480910389FE0FDC7F3400@BYAPR02MB5189.namprd02.prod.outlook.com> <AB059D94-9BCA-4794-BCD4-211D7E8E80F2@brianrosen.net> <BYAPR02MB5189BE4C40E7A72BCFC8BD03F35D0@BYAPR02MB5189.namprd02.prod.outlook.com> <959DCC43-1686-49D3-9195-719CF65C9EE9@brianrosen.net> <BYAPR02MB5189D7055FCD08B0F926B9ADF35A0@BYAPR02MB5189.namprd02.prod.outlook.com>
X-Mailer: Apple Mail (2.3608.120.23.2.1)
Archived-At: <https://mailarchive.ietf.org/arch/msg/stir/H7i2cCv9w5i53GDfnWepJzNV2n8>
Subject: Re: [stir] Review of draft-ietf-stir-rph-emergency-services
X-BeenThere: stir@ietf.org
X-Mailman-Version: 2.1.29
Precedence: list
List-Id: Secure Telephone Identity Revisited <stir.ietf.org>
List-Unsubscribe: <https://www.ietf.org/mailman/options/stir>, <mailto:stir-request@ietf.org?subject=unsubscribe>
List-Archive: <https://mailarchive.ietf.org/arch/browse/stir/>
List-Post: <mailto:stir@ietf.org>
List-Help: <mailto:stir-request@ietf.org?subject=help>
List-Subscribe: <https://www.ietf.org/mailman/listinfo/stir>, <mailto:stir-request@ietf.org?subject=subscribe>
X-List-Received-Date: Tue, 25 Aug 2020 15:15:04 -0000

Hi Jack,

I think we still want to keep “auth” and “ESorig/ESCallback” as separate and distinct key/value objects, i see your comments as maybe disagreement stylistically but didn’t see anything to suggest that it wouldn’t work.  The reasoning though is that “auth” is intended to be specific to priority services only, not extensible for other applications.  And the fact of the matter is key/values are cheap, no reason we can’t have new values and be explicit about the intent.  This document is specific to 911 and emergency services and to extend the key/values defined for ‘rph’ claim and ‘rph’ PASSporT extension.

I will make this clearer in the document and explicitly say that “auth” should not be used for emergency services purposes of using resource priority header.

I think it would also make sense to make it clear that that this document extends 8443 and linking the documents.

For compact form, we think it would confuse folks to add provisions for compact form or allow it for little gain, and therefore prefer to leave it undefined.

Thanks.

-Chris

> On Aug 20, 2020, at 1:30 PM, Jack Rickard <Jack.Rickard@metaswitch.com> wrote:
> 
> It's not clear that this spec relies on that, and I'm not sure that it should? To me it seems like this could be more widely applicable.
>  
> Would you be able to address, from a verification service's point of view, what it should do? I think that will clear up why "Esorig" and "EScallback" are useful above plain "auth".
> I think the spec would benefit from that section as well Section 4 of RFC 8443 and section 11 of draft-ietf-stir-passport-rcd-06 are both incredibly useful.
>  
> Thanks,
> Jack
>  
> From: Brian Rosen <br@brianrosen.net <mailto:br@brianrosen.net>> 
> Sent: 19 August 2020 14:40
> To: Jack Rickard <Jack.Rickard@metaswitch.com <mailto:Jack.Rickard@metaswitch.com>>
> Cc: draft-ietf-stir-rph-emergency-services@ietf.org <mailto:draft-ietf-stir-rph-emergency-services@ietf.org>; stir@ietf.org <mailto:stir@ietf.org>
> Subject: Re: [stir] Review of draft-ietf-stir-rph-emergency-services
>  
> NOTE: Message is from an external sender
> About 20% of the US has now upgraded to NG9-1-1, although most service providers haven’t yet.  The wireless carriers are moving towards it; testing is underway.  
> I will have a companion document in sipcore that handles the signaling aspects of this mechanism.  For the older E911 system, none of stir really helps because the existing system is built on the older Class 4 switches (“Selective Routers”).  All of this mechanism is for NG9-1-1 and the equivalent services in the rest of the world, based on IETF standards (RFC6881, RFC5222, and several others).  
> 
> 
> On Aug 19, 2020, at 7:19 AM, Jack Rickard <Jack.Rickard@metaswitch.com <mailto:Jack.Rickard@metaswitch.com>> wrote:
>  
> Thanks, that's really useful, however, I'm not convinced.
>  
> For ESorig, as far as I know this isn't currently always the case, I am aware of situations where emergency calls just look like normal calls with a number of 911. I'm also still unclear about what affect this has, as you can currently already sign emergency calls using the standard mechanisms (barring the whole "urn:service:sos" tn which I don't believe is standardised for STIR yet?)  by putting "esnet.0" in the auth field of an rph passport.
>  
> For EScallback, if the sph and "rph.auth" claims cover theis entirely why is EScallback needed? I'm also still unclear what the verification service behaviour is here.
>  
> I'm still unclear as to how this spec helps with allowing and preventing malicious use of emergency calls. You can already put esnet resource priority pairs in the "rph.auth" claim and that seems to provide as much security as this does. I do agree the sph claim does provide value, however.
>  
> I did just realise that the esnet Resource Priority needs to be in the "rph.auth" claim or the logic from that rfc will kick in and remove it, as per:
> RFC 8443, section 4.2
>    In such scenarios, the SIP 'Resource-Priority' header field SHOULD be
>    stripped from the SIP request, and the network entities should treat
>    the call as an ordinary call.
>  
> I'll note that some of my questions and concerns haven't been addressed yet, I'm happy to resolve this first, just making a note so that they don't get lost/I don't forget about them.
> Jack
>  
> From: Brian Rosen <br@brianrosen.net <mailto:br@brianrosen.net>> 
> Sent: 18 August 2020 22:31
> To: Jack Rickard <Jack.Rickard@metaswitch.com <mailto:Jack.Rickard@metaswitch.com>>
> Cc: draft-ietf-stir-rph-emergency-services@ietf.org <mailto:draft-ietf-stir-rph-emergency-services@ietf.org>; stir@ietf.org <mailto:stir@ietf.org>
> Subject: Re: [stir] Review of draft-ietf-stir-rph-emergency-services
>  
> Let’s keep the two use cases separate: 
> ESorig is an emergency call (user to authority).
> EScallback is a call from authority to user.
>  
> For ESorg, the call is marked with a Request-URI of urn:service:sos.  The “To” field is ignored in routing and handling the call.  That’s how you know this is an emergency call.   For stir purposes, the From is an ordinary identifier, and gets treated in the normal way.  However, the rph value can only be used by a valid emergency call, and a valid emergency call is a Request URI of urn:service:sos, and a valid From.  So you don’t have any knowledge of emergency services identifiers, only the Request URI of urn:service:sos.   
>  
> For EScallback, the marking is a distinguished value in SIP Priority.  If a call has that value, it’s a call back.  The originating service provider knows who are the authorities who are allowed to place call backs, so the check is what is in SIP Priority, and one of the allowed authorities in From.  As a practical matter, in most cases, the call will be signed by the emergency authorities themselves, who will be able to get appropriate credentials for this purpose.  In most cases I’m aware of, the To in an emergency call won’t match the From in a Callback, but it’s possible for that to happen, and we want to allow it.
>  
> Esnet is a Resource Priority Header namespace, not a SIP Priority value.  We’re allowing emergency calls and call backs to use rph, and we’re protecting against malicious use of it.  So unless the call has urn:service:sos in Request URI, or the right SIP Priority in a call back (and from an authorized authority) they can’t use rph with the esnet namespace.  Generally, unless under attack, emergency calls go through even if there isn’t a passport.  Many networks won’t offer actual priority to emergency calls, but the emergency services networks (ESInets) will.  
>  
> If a call arrives anywhere with an rph using esnet, and isn’t an emergency call, with the appropriate passport,  any intermediary can refuse to give it any priority,  The emergency authorities may accept a call without a valid passport, but they might treat it with much more suspicion than they would any other call.
>  
> For call backs, the network may have special behavior.  For example, it may send the call to the device that placed the emergency call, rather than, for example, voice mail.  The network has to be able to trust that SIP Priority is set appropriately.  It will also have rph using esnet, and the network can grant it appropriate priority if it has that capability, and would not allow esnet otherwise.
>  
> The only value in covering other uses of SIP Priority is to protect against middle boxes modifying it.  We don’t really have use cases for that.  It probably wouldn’t hurt to allow it though.
>  
> Does that help?
>  
> Brian
>  
> 
> 
> 
> On Aug 14, 2020, at 1:21 PM, Jack Rickard <Jack.Rickard=40metaswitch.com@dmarc.ietf.org <mailto:Jack.Rickard=40metaswitch.com@dmarc.ietf.org>> wrote:
>  
> I have reviewed draft-ietf-stir-rph-emergency-services-02 and have some concerns and questions. I don't believe this spec is implementable in its current form.
>  
> Thanks,
> Jack
>  
> Why are "ESorig" and "EScallback" distinct?
> They seem to serve a very similar purpose with the only difference being whether orig or dest should be the emergency services. I don't believe there's any check that can be done to validate:
>    When using "ESorig" as the "rph" assertion value, the "orig" claim of
>    the PASSporT MUST represent the calling party number that initiates
>    the call to emergency services.
> This (and the equivalent statement for EScallback) don't seem possible to me to check (barring the standard orig/dest checking)..
> This would make the check "at least one of the parties should be the emergency services" enough to validate that this was a reasonable call.
>  
>  
> Why have them at all rather than just using auth?
> This is very possibly an issue with my understanding, but I'm not clear on why "ESorig" and "EScallback" even need to exist. "esnet.0" etc. are SIP Resource Priority headers, so should be included in the "auth" field anyway by the RPH spec. This spec appears to apply extra constraints (that a one of the callers must be the emergency services) for the "esnet" namespace but it's not clear why entirely separate claims are needed.
>  
> In a similar vein, what should a verifier do if it receives a call containing an invalid "ESorig" or "EScallback" value or the passport is invalid/untrusted, I'm assuming the behaviour is the same as for auth but this isn't clear. Although stripping the fact that this is an emergency call is potentially dangerous..
>  
>  
> Specify the type of the "ESorig" and "EScallback" claims.
> This specification currently doesn't specify the type of the new fields, there are only examples and this isn't enough. It looks like they both follow the same scheme as the rph "auth" claim, however "esnet,x" doesn't quite fit into that due to the comma and that x isn't a valid priority value.
>  
>  
> Why is sph limited to psap-callback? What should the verifier do if it isn't that? What should it do if it is?
> I'm not entirely clear on the purpose of the sph claim, however, it seems odd that it doesn't cover the full range of possible values for the SIP Priority Header. Is there a reason that it doesn't cover the "non-urgent", "normal", "urgent", or "emergency" values?
> There is also no verifier behaviour defined here, should the verifier remove the Priority header if it receives an invite with no passports signing for it? That seems dangerous to me but would be consistent with rph. Alternatively, what should the verifier do if it receives an invite with a valid passport claiming sph but with no Priority header should it add it in? I'm not sure the spec needs to be too prescriptive here, however some mention of verifier behaviour and the associated security considerations would be useful.
> 
> 
> Should the spec be stronger about the compact form?
> Section 3 currently states
>    The use of the compact form of PASSporT is not specified in this
>    document.
> However, a compact-form passport following this spec would be hard to verify as it introduces multiple possible rph variants, I think this spec could go further and say you shouldn't/mustn't.
> 
> 
> What is the requirement of these new parameters.
> As I understand it, the passport spec allows you to create a passport containing whatever JWT fields you want and verifiers should just ignore any fields they don't understand. Unless the "ppt" claim is set, which indicates that verifiers should discard it if they don't recognise that passport type. As this spec adds additional fields to an existing passport type it isn't immediately clear what the behaviour should be. Specifically, is an rph passport containing only "rph.ESorig" valid now (where it wouldn't be before because auth isn't present), is an rph passport containing no rph and only sph valid, and what does a non-rph passport with sph or ESorig set mean?
>  
> _______________________________________________
> stir mailing list
> stir@ietf.org <mailto:stir@ietf.org>
> https://www.ietf.org/mailman/listinfo/stir <https://nam01.safelinks.protection.outlook.com/?url=https%3A%2F%2Fwww.ietf.org%2Fmailman%2Flistinfo%2Fstir&data=02%7C01%7CJack.Rickard%40metaswitch.com%7C2bc3d7e4e6aa4f85ebeb08d844457330%7C9d9e56ebf6134ddbb27bbfcdf14b2cdb%7C1%7C0%7C637334412375899805&sdata=jkZlyWrqCNpbqPArsBZNryW%2BWjTylJY%2FH86b1bwMLDQ%3D&reserved=0>