Return-Path: <rifaat.s.ietf@gmail.com>
X-Original-To: oauth@mail2.ietf.org
Delivered-To: oauth@mail2.ietf.org
Received: from localhost (localhost [127.0.0.1])
	by mail2.ietf.org (Postfix) with ESMTP id D4966E2FB5DC
	for <oauth@mail2.ietf.org>; Sat, 25 Apr 2026 07:35:00 -0700 (PDT)
DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/simple; d=ietf.org; s=ietf1;
	t=1777127700; bh=wbE6GVDiqMsV9Ncb5ggGfiwdIbrMTjfvGgPAhBtsVgw=;
	h=References:In-Reply-To:From:Date:Subject:To:Cc;
	b=KUpGb2iOLEQKtIMuSZNZph9yA/ORg+jhjEb7LeLT/8qhMqwEC4A/JotV2H6ymoeBF
	 dZtTmnfGLrvoRlKnti9zBHbQO2ciMJevRGNDEkk+C5HEjSEacgqBPEPfXaVXQbIB0I
	 p4UQCYWyIexrZRhlW+9z6msAZBo4RvPrOFzTqUpM=
X-Virus-Scanned: amavisd-new at ietf.org
X-Spam-Flag: NO
X-Spam-Score: -2.098
X-Spam-Level: 
X-Spam-Status: No, score=-2.098 tagged_above=-999 required=5
	tests=[BAYES_00=-1.9, DKIM_SIGNED=0.1, DKIM_VALID=-0.1,
	DKIM_VALID_AU=-0.1, DKIM_VALID_EF=-0.1, FREEMAIL_FROM=0.001,
	HTML_MESSAGE=0.001, RCVD_IN_DNSWL_NONE=-0.0001, SPF_HELO_NONE=0.001,
	SPF_PASS=-0.001] autolearn=ham autolearn_force=no
Authentication-Results: mail2.ietf.org (amavisd-new); dkim=pass (2048-bit key)
	header.d=gmail.com
Received: from mail2.ietf.org ([166.84.6.31])
	by localhost (mail2.ietf.org [127.0.0.1]) (amavisd-new, port 10024)
	with ESMTP id WiC6_P3hQupq for <oauth@mail2.ietf.org>;
	Sat, 25 Apr 2026 07:35:00 -0700 (PDT)
Received: from mail-lf1-x134.google.com (mail-lf1-x134.google.com
 [IPv6:2a00:1450:4864:20::134])
	(using TLSv1.3 with cipher TLS_AES_128_GCM_SHA256 (128/128 bits)
	 key-exchange X25519 server-signature ECDSA (P-256) server-digest SHA256)
	(No client certificate requested)
	by mail2.ietf.org (Postfix) with ESMTPS id 006D6E2FB5CD
	for <oauth@ietf.org>; Sat, 25 Apr 2026 07:34:59 -0700 (PDT)
Received: by mail-lf1-x134.google.com with SMTP id
 2adb3069b0e04-5a62a049c1fso6029408e87.3
        for <oauth@ietf.org>; Sat, 25 Apr 2026 07:34:59 -0700 (PDT)
ARC-Seal: i=1; a=rsa-sha256; t=1777127699; cv=none;
        d=google.com; s=arc-20240605;
        b=U6JUeY4OAylBM41zRZDiAN0MqyV8L1Tf0PIn1D+4d7Q0zT0UuZavA/OrbfRHrq2HPJ
         /NgF3aaQa7HplpXL918fjr4svsFF2yM47rUpj1WfuZRocgGxo9UmOpzh5XjkvvpwRdtn
         Z92TKmVMZhvzQoYtchM0xAbBu/TC25xbDVr10eTkyMQuPH8lsPXow2rdY6PC+3owgYKd
         u/8F5xwZ5wRpcLpqBTN8Ol1GZC/GvnfdTqmmBabglVGgUpugpzMi68kk0h0pgNaxf2lr
         18/y+Zn727nefqAISLSerz6vHm4tuOX0S1qHV+46LB14dDAPA1NELwvuKM9BHXhsuu5C
         caiQ==
ARC-Message-Signature: i=1; a=rsa-sha256; c=relaxed/relaxed; d=google.com;
 s=arc-20240605;
        h=cc:to:subject:message-id:date:from:in-reply-to:references
         :mime-version:dkim-signature;
        bh=njk1/oxrv4a79JVuDAJz5FE/paaH/xqch4uIb4jTtas=;
        fh=P+nKllo/JihWfgLYquSFzcJSo9gMon8bVKzsCG/4zIM=;
        b=dLDD8DJJphp7v8Nd7f3gVlL3p9TPmF5zdgTQr8tZZkW1dKOcGmsK7t/bBNlYIjzRT7
         oNwfoiD97slM3OR/92dXiYbu1YfCAxsBvlMXXZ2WJl+eH7NwcHdEQpL+78h76xYFFKDW
         XjnkVfpc3cNtbwdLGZsXdhjKCafMD5GlJFWSn/wsginjcsElH4SjhUJtHeZwhYd8kVKF
         ZPuvsvAs31pmORxp3hNduC0l3805n7u6StHNLs7Mh+aHUcawZLV+w0nlhzk9cEVJ7vqk
         QoOLihj/XyUAKlol3ABp9TeI6M7lPIKHKh3276lBmh0geQDeK6FNp1SHf/ciEfSWtj9K
         wFlQ==;
        darn=ietf.org
ARC-Authentication-Results: i=1; mx.google.com; arc=none
DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed;
        d=gmail.com; s=20251104; t=1777127699; x=1777732499; darn=ietf.org;
        h=cc:to:subject:message-id:date:from:in-reply-to:references
         :mime-version:from:to:cc:subject:date:message-id:reply-to;
        bh=njk1/oxrv4a79JVuDAJz5FE/paaH/xqch4uIb4jTtas=;
        b=OUs4FmjzRUS0I3jnjAlb+8hDWJqxv2V/m1OBpZnC1qsbfGLHslHuWGnr+93CctSghI
         loBfbnBQ02dfjsdoMJOYf9a+8NtrrZXWCNrqlkQ3zpYMP4cI5ig4SRvBXuiPLjsbsHzy
         FIpTCpJD2mBUjIszJE9i5zY0TogOCNYzAiQwd4HVx8fOW7IPdU6r26S7Dv0dLxDnDHOm
         aokgKzMT7R61abv+IdGdm2tc2dihD38KXo8oTrOrYJUYWeG4mNze29IPUK/vQNlNnE8g
         yLA6jOlG2kROPophAaqiwYv8zrM8MQEuIW1ErJRrfpuBveQsMONlBHpSDNPi9gO4sFyB
         PcFA==
X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed;
        d=1e100.net; s=20251104; t=1777127699; x=1777732499;
        h=cc:to:subject:message-id:date:from:in-reply-to:references
         :mime-version:x-gm-gg:x-gm-message-state:from:to:cc:subject:date
         :message-id:reply-to;
        bh=njk1/oxrv4a79JVuDAJz5FE/paaH/xqch4uIb4jTtas=;
        b=r030mxmJCib6W+Jao/2GzjHey1vQ4dbQ282K+mfkb/H4SEfl/RpppWTAxNZHd1FabW
         AifxRtOgHEDy91G4JTaT/fDs9tXDzC3zbVRwfJ7hGQaAvoL/ER0OgCjTz9JYvekWYyD0
         TMObBCdkqXPdK9FeleTLI5OboeijH8TsfFRBTV10ZP+JUcPsGSnXZyGnXMJC/33wM719
         tW44Jz/4OMP71oly9KnRfkgH++usYwMtmmqENFgj48ngC96SSW+DHzEB2AYwciK3iOMw
         ZaaE3qSlH/BJeBFPBgeC5+3JvBZ3s3JzTYztW8SF3wojp0ma+SFel9J5RP57dEuJsO4g
         coQg==
X-Forwarded-Encrypted: i=1;
 AFNElJ9EjdtIW6lsx1Q7fq0WeCdL/3AdW1cg22tX1FnXQ5ZboT0mz5YKHNhKg4rLgvk0oTYJ24ZXog==@ietf.org
X-Gm-Message-State: AOJu0YyVwl9QZuSeqTzMGu4sZnTGxkhGb5fJvxfK1A3vSToKX6r0uwhe
	oM3eElNIezsnDcGOG8yUXatRd//6beGxyDyAuefGEFltlSK+IhIA17fDdKzGxRzi6s1yi1/FPyP
	/S6vWq3HCXH5+TrLEev9QS6UcE7xOha5+BquB
X-Gm-Gg: AeBDieuI3O24fyrJW1LPKUfyH9OklgetJnbLpaVVRRm6PMQV7pfGhbGayVNz81HKD/X
	1bPu5u3vZ140MsWMJ/7841AKFbfHcmoiecOLOe/QruAlvbWKwDFxb7o9c+d49tBB57DLcH7GJgC
	3I/1RQ3cx/EgLySYJUD/RwYk7K8jz6SZAOBtnHueLZ4PkTKk9YxfiFh2rAo+GWuptetq3E302q8
	xo5TgMMpto43/7AAjnVLmsLYNzByDuxv3+YtRROn5IauE0vwzYS+Dg3nhMzUvJnzDO2Uh80W06s
	FGkQiUe+WIE2E1UVg4B31+cGPi/OROrTXFwK8KLYUqgnAMc/QNYmm2zxjQbfiZN23R8kH1I/dTs
	3Z6va0/hibwMskTF2gm92yafU/dJDuibFibrOXA==
X-Received: by 2002:a05:6512:3510:b0:5a4:194f:5fff with SMTP id
 2adb3069b0e04-5a4194f60edmr8792932e87.16.1777127698447; Sat, 25 Apr 2026
 07:34:58 -0700 (PDT)
MIME-Version: 1.0
References: 
 <CAGgd1Od5Bx9NkDwMgW7LgC6YaCpdm_DHYXStD1K4dprybeK6YA@mail.gmail.com>
 <CA+k3eCQiUfK1ytknu9qm+B_hKaqqvK8652ZDkewQMubUm+kkcw@mail.gmail.com>
 <CAGgd1OcotkgZOYtD_zFEe51j7NXSZ=ZXzaKfD=9erXdM2_grQg@mail.gmail.com>
In-Reply-To: 
 <CAGgd1OcotkgZOYtD_zFEe51j7NXSZ=ZXzaKfD=9erXdM2_grQg@mail.gmail.com>
From: Rifaat Shekh-Yusef <rifaat.s.ietf@gmail.com>
Date: Sat, 25 Apr 2026 10:34:47 -0400
X-Gm-Features: AVHnY4LURBK8V9qdOISEQGLWKJ-7uwbHd4IKh38iJOFcq_9-bbY2V-KqYsPkadI
Message-ID: 
 <CADNypP94_PhyiFNMq-0Jh-nk8r=aC5vmNLuBLFRs1WKy9rcJAA@mail.gmail.com>
To: Deb Cooley <debcooley1@gmail.com>
Content-Type: multipart/alternative; boundary="0000000000002fc2aa065049c9a2"
Message-ID-Hash: 4NFN7OTG72M5DBQBUMU2C3YN4LSGB3LO
X-Message-ID-Hash: 4NFN7OTG72M5DBQBUMU2C3YN4LSGB3LO
X-MailFrom: rifaat.s.ietf@gmail.com
X-Mailman-Rule-Misses: dmarc-mitigation; no-senders; approved; emergency;
 loop; banned-address; member-moderation; header-match-oauth.ietf.org-0;
 nonmember-moderation; administrivia; implicit-dest; max-recipients; max-size;
 news-moderation; no-subject; digests; suspicious-header
CC: draft-ietf-oauth-identity-chaining.authors@ietf.org,
 Web Authorization Protocol Working Group <oauth-chairs@ietf.org>,
 oauth <oauth@ietf.org>
X-Mailman-Version: 3.3.9rc6
Precedence: list
Subject: =?utf-8?q?=5BOAUTH-WG=5D_Re=3A_AD_comments_on_draft-ietf-oauth-identity-chai?=
	=?utf-8?q?ning?=
List-Id: OAUTH WG <oauth.ietf.org>
Archived-At: 
 <https://mailarchive.ietf.org/arch/msg/oauth/KIHMvqQcfSW3ctwMvig6C1koixY>
List-Archive: <https://mailarchive.ietf.org/arch/browse/oauth>
List-Help: <mailto:oauth-request@ietf.org?subject=help>
List-Owner: <mailto:oauth-owner@ietf.org>
List-Post: <mailto:oauth@ietf.org>
List-Subscribe: <mailto:oauth-join@ietf.org>
List-Unsubscribe: <mailto:oauth-leave@ietf.org>

--0000000000002fc2aa065049c9a2
Content-Type: text/plain; charset="UTF-8"
Content-Transfer-Encoding: quoted-printable

On Sat, Apr 25, 2026 at 9:31=E2=80=AFAM Deb Cooley <debcooley1@gmail.com> w=
rote:

> These all look good to me.  I'm going to submit it to IETF Last Call.
>
> Adding Aaron as an author is fine as well. I will need Rifaat to update
> the shepherd review to provide at least a little bit of rationale for 6
> authors.
>

Done

Regards,
 Rifaat


>
> Other comments/confessions?  in line with [DC]
>
> Deb
>
> On Fri, Apr 24, 2026 at 12:11=E2=80=AFPM Brian Campbell <
> bcampbell@pingidentity.com> wrote:
>
>> Thank you for the review and it's now my turn to apologize for the delay
>> (can I blame IETF 125 at this point? probably not...).  Some specific
>> responses are inline below but I don't think you'll find anything
>> surprising or controversial.  This issue
>> https://github.com/oauth-wg/oauth-identity-chaining/issues/182 and this
>> PR https://github.com/oauth-wg/oauth-identity-chaining/pull/183 kinda
>> show our work here and the responses below are taken from the issue and
>> specific commits mentioned are part of that PR. A new draft incorporatin=
g
>> all this will be forthcoming soon.
>>
>>
>> On Wed, Apr 8, 2026 at 6:57=E2=80=AFAM Deb Cooley <debcooley1@gmail.com>=
 wrote:
>>
>>> Thank you for your work on this specification, here are my comments on
>>> this draft.  Apologies for the delay (IETF 125 and recovery from the sa=
me
>>> takes time).  I don't think any of these comments are especially diffic=
ult,
>>> most are process related:
>>>
>>> Deb
>>> Sec AD
>>>
>>> ---------------------------------------
>>> General:  Consider using *.*.example.org for example domains (currently
>>> this draft uses as.*.org).  See:
>>> https://authors.ietf.org/example-addresses  for more information.
>>>
>>
>> Yup, will do in 3c777e3
>> <https://github.com/oauth-wg/oauth-identity-chaining/commit/3c777e33338d=
8f559bec358d5008bad801ec3d7c>
>> and 886cbc6
>> <https://github.com/oauth-wg/oauth-identity-chaining/commit/886cbc6f71b3=
4dffa41064f9ef9f14aef2eb84e4>
>>
> [DC] certainly the change results in a 'non-routable' domain, but conside=
r
> adding '.org' to the end.  This can wait for later, and possibly for
> someone else commenting on it.
>
>>
>>
>>
>>> Long line:  Line 416, is 75 characters long, according to idnits
>>> (experimental).  I did not go looking for this (which is why I've put i=
t up
>>> front).
>>>
>>
>> will split the 1 instance of too long line
>> <https://github.com/oauth-wg/oauth-identity-chaining/pull/183/changes/1f=
32fc4af6b5efb85aca27e82183cc4301b3dcee>
>>
>>
>>
>>> Section 2.1, Appendix B:  Using steps (A), (B), (C) could be confusing
>>> in light of the pre-existing domain enumerations of Domain A, Domain B.
>>> I'd suggest using some other enumeration for the steps, maybe (1), (2),=
 (3)
>>> to disambiguate.
>>>
>>
>> Makes good sense 7a90229
>> <https://github.com/oauth-wg/oauth-identity-chaining/commit/7a9022977217=
5b040c487982637cb0b94cc32661>
>>
>>
>> Section 2.2:  Since this 'MAY' does not enable/disable interoperability
>>> (the goal of BCP 14), consider changing it to 'may' or even 'could'.
>>>
>>
>> Happy to do that one cf9e53f
>> <https://github.com/oauth-wg/oauth-identity-chaining/commit/cf9e53fa063e=
7814ceccf7622947d138244221a4>
>>
>>
>>
>>> Section 2.3, bullet 1:  'MUST deny' or 'MUST NOT approve'?  Which will
>>> be more obvious to the implementer?
>>>
>>
>> In bullet 1 or 2.3[.2?], deny seems more obvious to me and I was an
>> implementer at one point long ago in this so called career.
>>
>> [DC] this is fine.
>
>>
>>
>>>
>>> Section 2.4.3:  There is an assumption here that the previous
>>> rules/steps/checks have been validated without error.  Perhaps consider
>>> adding a phrase like, 'When the authorization grant has been validated,
>>> the....'
>>>
>>
>> Yup, added When the authorization grant has been validated, the...
>> <https://github.com/oauth-wg/oauth-identity-chaining/pull/183/changes/7d=
f77d37938e3bab958a7e5426b3e5cb0f26d593>
>>
>>
>>
>>> Section 5:  BCP14 language or not?  Please see:
>>> https://datatracker.ietf.org/doc/statement-iesg-statement-on-clarifying=
-the-use-of-bcp-14-key-words/
>>> . While BCP 14 words in Security Considerations is completely appropria=
te,
>>> it should still be used sparingly.  The word 'must' is just as normativ=
e as
>>> 'MUST', but carries less baggage.  I would suggest going back through
>>> Section 5, and consider which of the BCP 14 words are there for
>>> 'interoperability' or to 'prevent harm'.  Please pay special attention =
to
>>> section on 'SHOULD' and 'RECOMMENDED'.  I'm making some comments within
>>> Section 5 to emphasize my point of view.
>>>
>>> Section 5.1:  Naked 'SHOULD's need to have a 'what are the consequences
>>> of violating the should' that goes with it.  An alternative is to state
>>> under what reasonable circumstances the should isn't possible (maybe th=
at
>>> is the public client part of this section?).
>>>
>>> Section 5.4:  I'm guessing that there was confusion at some point about
>>> issuance of refresh tokens by AS domain A to clients in domain A?  My
>>> suggestion to make this a tiny bit clearer in para 1 is to add 'domain =
A'
>>> when you mention 'client' -> 'client in domain A'.  It does make it wor=
dier
>>> and longer, but way more obvious that the refresh tokens across domains=
 are
>>> the issue.
>>>
>>> Section 5.4: It is also not clear why one might choose to violate the
>>> sequence of 'SHOULD NOT', 'SHOULD', 'SHOULD'...  Why not 'MUST NOT',
>>> 'MUST', 'MUST'?  Sadly, RFC 7521 doesn't specify why/when 'SHOULD NOT'
>>> could be ignored (unless I read it incorrectly).
>>>
>>> Section 5.5:  Why not 'MUST'?  Seems reasonable to say that one MUST
>>> evaluate the risk and since there is no definition of 'appropriate', th=
en
>>> any mitigations will do, no?
>>>
>>
>> All of this makes sense and I took a pass over sec 5 with it in mind Sec=
urity
>> Considerations: attempt improvements around normative language and other
>> language too
>> <https://github.com/oauth-wg/oauth-identity-chaining/pull/183/changes/2a=
c9612d7dded24e55dc8e292448d9ed3cc5167b>
>>
>> [DC] these all look good.
>
>>
>>
>>>
>>> IANA Considerations:  Add this, see:
>>> https://authors.ietf.org/en/iana-considerations .  It *might* be
>>> possible to remove it prior to publication.  IANA will know (and tell y=
ou)
>>> for sure.
>>>
>>
>> This led me to look at and makes some small changes with Fiddling with
>> IANA Considerations
>> <https://github.com/oauth-wg/oauth-identity-chaining/pull/183/changes/92=
2023acd58aa19deb77d6fbbf7c32afa6a09759>
>> but otherwise I'm not sure what action or learning I was supposed to tak=
e
>> from that?
>>
>>
> [DC] sigh.... it was the placement of this section that confused me.  No
> problem, I just didn't win the game of hide and seek the IANA Considerati=
on
> section game (this is me poking fun at myself, no changes required).
>
>>
>>
>>> Acknowledgements (Appendix C):  I would remove the parenthetical and
>>> just state a generic 'and others for their valuable input....'.  Naming
>>> everyone who had input is not actually required.  [rationale:  the () w=
on't
>>> age well]
>>>
>>
>> The parenthetical was really meant as an invitation to people that we ma=
y
>> have overlooked to let us know so we could include them. But the time fo=
r
>> that has passed so we'll take it out. 2824b2e
>> <https://github.com/oauth-wg/oauth-identity-chaining/commit/2824b2e3dd0c=
536ff09879ed42ca7f013b68d44f>
>>
>>
>> *CONFIDENTIALITY NOTICE: This email may contain confidential and
>> privileged material for the sole use of the intended recipient(s). Any
>> review, use, distribution or disclosure by others is strictly prohibited=
.
>> If you have received this communication in error, please notify the send=
er
>> immediately by e-mail and delete the message and any file attachments fr=
om
>> your computer. Thank you.*
>
>

--0000000000002fc2aa065049c9a2
Content-Type: text/html; charset="UTF-8"
Content-Transfer-Encoding: quoted-printable

<div dir=3D"ltr"><div dir=3D"ltr"><br></div><br><div class=3D"gmail_quote g=
mail_quote_container"><div dir=3D"ltr" class=3D"gmail_attr">On Sat, Apr 25,=
 2026 at 9:31=E2=80=AFAM Deb Cooley &lt;<a href=3D"mailto:debcooley1@gmail.=
com">debcooley1@gmail.com</a>&gt; wrote:<br></div><blockquote class=3D"gmai=
l_quote" style=3D"margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,20=
4,204);padding-left:1ex"><div dir=3D"ltr"><div dir=3D"ltr"><div>These all l=
ook good to me.=C2=A0 I&#39;m going to submit it to IETF Last Call.</div><d=
iv><br></div><div>Adding Aaron as an author is fine as well. I will need Ri=
faat to update the shepherd review to provide at least a little bit of rati=
onale for 6 authors.</div></div></div></blockquote><div><br></div><div>Done=
</div><div><br></div><div>Regards,</div><div>=C2=A0Rifaat</div><div>=C2=A0<=
/div><blockquote class=3D"gmail_quote" style=3D"margin:0px 0px 0px 0.8ex;bo=
rder-left:1px solid rgb(204,204,204);padding-left:1ex"><div dir=3D"ltr"><di=
v dir=3D"ltr"><div><br></div><div>Other comments/confessions?=C2=A0 in line=
 with [DC]</div><div><br></div><div>Deb</div></div><br><div class=3D"gmail_=
quote"><div dir=3D"ltr" class=3D"gmail_attr">On Fri, Apr 24, 2026 at 12:11=
=E2=80=AFPM Brian Campbell &lt;<a href=3D"mailto:bcampbell@pingidentity.com=
" target=3D"_blank">bcampbell@pingidentity.com</a>&gt; wrote:<br></div><blo=
ckquote class=3D"gmail_quote" style=3D"margin:0px 0px 0px 0.8ex;border-left=
:1px solid rgb(204,204,204);padding-left:1ex"><div dir=3D"ltr"><div>Thank y=
ou for the review and it&#39;s now my turn to apologize=C2=A0for the delay =
(can I blame IETF 125 at this point? probably not...).=C2=A0 Some specific =
responses are inline below but I don&#39;t think you&#39;ll find anything s=
urprising or controversial.=C2=A0 This issue <a href=3D"https://github.com/=
oauth-wg/oauth-identity-chaining/issues/182" target=3D"_blank">https://gith=
ub.com/oauth-wg/oauth-identity-chaining/issues/182</a> and this PR <a href=
=3D"https://github.com/oauth-wg/oauth-identity-chaining/pull/183" target=3D=
"_blank">https://github.com/oauth-wg/oauth-identity-chaining/pull/183</a> k=
inda show our work here and the responses below are taken from the issue an=
d specific commits mentioned are part of that PR. A new draft incorporating=
 all this will be forthcoming soon.=C2=A0</div><div dir=3D"ltr"><br></div><=
div><br></div><div class=3D"gmail_quote"><div dir=3D"ltr" class=3D"gmail_at=
tr">On Wed, Apr 8, 2026 at 6:57=E2=80=AFAM Deb Cooley &lt;<a href=3D"mailto=
:debcooley1@gmail.com" target=3D"_blank">debcooley1@gmail.com</a>&gt; wrote=
:<br></div><blockquote class=3D"gmail_quote" style=3D"margin:0px 0px 0px 0.=
8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex"><div dir=3D"lt=
r"><div>Thank you for your work on this specification, here are my comments=
 on this draft.=C2=A0 Apologies for the delay (IETF 125 and recovery from t=
he same takes time).=C2=A0 I don&#39;t think any of these comments are espe=
cially difficult, most are process related:=C2=A0</div><div><br></div><div>=
Deb</div><div>Sec AD</div><div><br></div><div>-----------------------------=
----------</div><div>General: =C2=A0Consider using *.*.<a href=3D"http://ex=
ample.org" target=3D"_blank">example.org</a> for example domains (currently=
 this draft uses as.*.org).=C2=A0 See: =C2=A0<a href=3D"https://authors.iet=
f.org/example-addresses" target=3D"_blank">https://authors.ietf.org/example=
-addresses</a> =C2=A0for more information.<br></div></div></blockquote><div=
><br></div><div>Yup, will do in <a href=3D"https://github.com/oauth-wg/oaut=
h-identity-chaining/commit/3c777e33338d8f559bec358d5008bad801ec3d7c" target=
=3D"_blank"><tt>3c777e3</tt></a> and <a href=3D"https://github.com/oauth-wg=
/oauth-identity-chaining/commit/886cbc6f71b34dffa41064f9ef9f14aef2eb84e4" t=
arget=3D"_blank"><tt>886cbc6</tt></a></div></div></div></blockquote><div>[D=
C] certainly the change results in a &#39;non-routable&#39; domain, but con=
sider adding &#39;.org&#39; to the end.=C2=A0 This can wait for later, and =
possibly for=C2=A0</div><div>someone else commenting on it.=C2=A0 =C2=A0</d=
iv><blockquote class=3D"gmail_quote" style=3D"margin:0px 0px 0px 0.8ex;bord=
er-left:1px solid rgb(204,204,204);padding-left:1ex"><div dir=3D"ltr"><div =
class=3D"gmail_quote"><div>=C2=A0</div><div><br></div><blockquote class=3D"=
gmail_quote" style=3D"margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(20=
4,204,204);padding-left:1ex"><div dir=3D"ltr"><div><br>Long line: =C2=A0Lin=
e 416, is 75 characters long, according to idnits (experimental).=C2=A0 I d=
id not go looking for this (which is why I&#39;ve put it up front). =C2=A0<=
br></div></div></blockquote><div><br></div><div>will <a href=3D"https://git=
hub.com/oauth-wg/oauth-identity-chaining/pull/183/changes/1f32fc4af6b5efb85=
aca27e82183cc4301b3dcee" target=3D"_blank">split the 1 instance of too long=
 line</a></div><div>=C2=A0</div><div><br></div><blockquote class=3D"gmail_q=
uote" style=3D"margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,2=
04);padding-left:1ex"><div dir=3D"ltr"><div><br>Section 2.1, Appendix B: =
=C2=A0Using steps (A), (B), (C) could be confusing in light of the pre-exis=
ting domain enumerations of Domain A, Domain B.=C2=A0 I&#39;d suggest using=
 some other enumeration for the steps, maybe (1), (2), (3) to disambiguate.=
</div></div></blockquote><div><br></div><div>Makes good sense <a href=3D"ht=
tps://github.com/oauth-wg/oauth-identity-chaining/commit/7a90229772175b040c=
487982637cb0b94cc32661" target=3D"_blank"><tt>7a90229</tt></a></div><div>=
=C2=A0</div><div><br></div><blockquote class=3D"gmail_quote" style=3D"margi=
n:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex=
"><div dir=3D"ltr"><div>Section 2.2: =C2=A0Since this &#39;MAY&#39; does no=
t enable/disable interoperability (the goal of BCP 14), consider changing i=
t to &#39;may&#39; or even &#39;could&#39;. <br></div></div></blockquote><d=
iv><br></div><div>Happy to do that one <a href=3D"https://github.com/oauth-=
wg/oauth-identity-chaining/commit/cf9e53fa063e7814ceccf7622947d138244221a4"=
 target=3D"_blank"><tt>cf9e53f</tt></a></div><div>=C2=A0</div><div><br></di=
v><blockquote class=3D"gmail_quote" style=3D"margin:0px 0px 0px 0.8ex;borde=
r-left:1px solid rgb(204,204,204);padding-left:1ex"><div dir=3D"ltr"><div><=
br>Section 2.3, bullet 1: =C2=A0&#39;MUST deny&#39; or &#39;MUST NOT approv=
e&#39;?=C2=A0 Which will be more obvious to the implementer?<br></div></div=
></blockquote><div><br></div><div>In bullet 1 or 2.3[.2?], deny seems more =
obvious to me and I was an implementer at one point long ago in this so cal=
led career.</div><div><br></div></div></div></blockquote><div>[DC] this is =
fine.=C2=A0=C2=A0</div><blockquote class=3D"gmail_quote" style=3D"margin:0p=
x 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex"><d=
iv dir=3D"ltr"><div class=3D"gmail_quote"><div></div><div>=C2=A0</div><bloc=
kquote class=3D"gmail_quote" style=3D"margin:0px 0px 0px 0.8ex;border-left:=
1px solid rgb(204,204,204);padding-left:1ex"><div dir=3D"ltr"><div><br>Sect=
ion 2.4.3: =C2=A0There is an assumption here that the previous rules/steps/=
checks have been validated without error.=C2=A0 Perhaps consider adding a p=
hrase like, &#39;When the authorization grant has been validated, the....&#=
39;<br></div></div></blockquote><div><br></div><div>Yup, added <a href=3D"h=
ttps://github.com/oauth-wg/oauth-identity-chaining/pull/183/changes/7df77d3=
7938e3bab958a7e5426b3e5cb0f26d593" target=3D"_blank">When the authorization=
 grant has been validated, the...</a></div><div>=C2=A0</div><div><br></div>=
<blockquote class=3D"gmail_quote" style=3D"margin:0px 0px 0px 0.8ex;border-=
left:1px solid rgb(204,204,204);padding-left:1ex"><div dir=3D"ltr"><div><br=
>Section 5: =C2=A0BCP14 language or not?=C2=A0 Please see: =C2=A0<a href=3D=
"https://datatracker.ietf.org/doc/statement-iesg-statement-on-clarifying-th=
e-use-of-bcp-14-key-words/" target=3D"_blank">https://datatracker.ietf.org/=
doc/statement-iesg-statement-on-clarifying-the-use-of-bcp-14-key-words/</a>=
 . While BCP 14 words in Security Considerations is completely appropriate,=
 it should still be used sparingly.=C2=A0 The word &#39;must&#39; is just a=
s normative as &#39;MUST&#39;, but carries less baggage.=C2=A0 I would sugg=
est going back through Section 5, and consider which of the BCP 14 words ar=
e there for &#39;interoperability&#39; or to &#39;prevent harm&#39;.=C2=A0 =
Please pay special attention to section on &#39;SHOULD&#39; and &#39;RECOMM=
ENDED&#39;.=C2=A0 I&#39;m making some comments within Section 5 to emphasiz=
e my point of view.<br><br>Section 5.1: =C2=A0Naked &#39;SHOULD&#39;s need =
to have a &#39;what are the consequences of violating the should&#39; that =
goes with it.=C2=A0 An alternative is to state under what reasonable circum=
stances the should isn&#39;t possible (maybe that is the public client part=
 of this section?).<br><br>Section 5.4: =C2=A0I&#39;m guessing that there w=
as confusion at some point about issuance of refresh tokens by AS domain A =
to clients in domain A?=C2=A0 My suggestion to make this a tiny bit clearer=
 in para 1 is to add &#39;domain A&#39; when you mention &#39;client&#39; -=
&gt; &#39;client in domain A&#39;.=C2=A0 It does make it wordier and longer=
, but way more obvious that the refresh tokens across domains are the issue=
. =C2=A0<br><br>Section 5.4: It is also not clear why one might choose to v=
iolate the sequence of &#39;SHOULD NOT&#39;, &#39;SHOULD&#39;, &#39;SHOULD&=
#39;...=C2=A0 Why not &#39;MUST NOT&#39;, &#39;MUST&#39;, &#39;MUST&#39;?=
=C2=A0 Sadly, RFC 7521 doesn&#39;t specify why/when &#39;SHOULD NOT&#39; co=
uld be ignored (unless I read it incorrectly).<br><br>Section 5.5: =C2=A0Wh=
y not &#39;MUST&#39;?=C2=A0 Seems reasonable to say that one MUST evaluate =
the risk and since there is no definition of &#39;appropriate&#39;, then an=
y mitigations will do, no?<br></div></div></blockquote><div><br></div><div>=
All of this makes sense and I took a pass over sec 5 with it in mind <a hre=
f=3D"https://github.com/oauth-wg/oauth-identity-chaining/pull/183/changes/2=
ac9612d7dded24e55dc8e292448d9ed3cc5167b" target=3D"_blank">Security Conside=
rations: attempt improvements around normative language and other language =
too</a></div><div><br></div></div></div></blockquote><div>[DC] these all lo=
ok good.=C2=A0=C2=A0</div><blockquote class=3D"gmail_quote" style=3D"margin=
:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex"=
><div dir=3D"ltr"><div class=3D"gmail_quote"><div></div><div>=C2=A0</div><b=
lockquote class=3D"gmail_quote" style=3D"margin:0px 0px 0px 0.8ex;border-le=
ft:1px solid rgb(204,204,204);padding-left:1ex"><div dir=3D"ltr"><div><br>I=
ANA Considerations: =C2=A0Add this, see: <a href=3D"https://authors.ietf.or=
g/en/iana-considerations" target=3D"_blank">https://authors.ietf.org/en/ian=
a-considerations</a> .=C2=A0 It *might* be possible to remove it prior to p=
ublication.=C2=A0 IANA will know (and tell you) for sure.<br></div></div></=
blockquote><div><br></div><div>This led me to look at and makes some small =
changes with <a href=3D"https://github.com/oauth-wg/oauth-identity-chaining=
/pull/183/changes/922023acd58aa19deb77d6fbbf7c32afa6a09759" target=3D"_blan=
k">Fiddling with IANA Considerations</a> but otherwise I&#39;m not sure wha=
t action or learning I was supposed to take from that?</div><div>=C2=A0</di=
v></div></div></blockquote><div>[DC] sigh.... it was the placement of this =
section that confused me.=C2=A0 No problem, I just didn&#39;t win the game =
of hide and seek the IANA Consideration section game (this is me poking fun=
 at myself, no changes required).=C2=A0</div><blockquote class=3D"gmail_quo=
te" style=3D"margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204=
);padding-left:1ex"><div dir=3D"ltr"><div class=3D"gmail_quote"><div><br></=
div><blockquote class=3D"gmail_quote" style=3D"margin:0px 0px 0px 0.8ex;bor=
der-left:1px solid rgb(204,204,204);padding-left:1ex"><div dir=3D"ltr"><div=
><br>Acknowledgements (Appendix C): =C2=A0I would remove the parenthetical =
and just state a generic &#39;and others for their valuable input....&#39;.=
=C2=A0 Naming everyone who had input is not actually required. =C2=A0[ratio=
nale: =C2=A0the () won&#39;t age well]=C2=A0=C2=A0</div></div></blockquote>=
<div><br></div><div>The parenthetical was really meant as an invitation to =
people that we=20
may have overlooked to let us know so we could include them. But the=20
time for that has passed so we&#39;ll take it out.  <a href=3D"https://gith=
ub.com/oauth-wg/oauth-identity-chaining/commit/2824b2e3dd0c536ff09879ed42ca=
7f013b68d44f" target=3D"_blank"><tt>2824b2e</tt></a>=C2=A0</div></div></div=
>

<br>
<i style=3D"margin:0px;padding:0px;border:0px;outline:0px;vertical-align:ba=
seline;background:rgb(255,255,255);font-family:proxima-nova-zendesk,system-=
ui,-apple-system,system-ui,&quot;Segoe UI&quot;,Roboto,Oxygen-Sans,Ubuntu,C=
antarell,&quot;Helvetica Neue&quot;,Arial,sans-serif;color:rgb(85,85,85)"><=
span style=3D"margin:0px;padding:0px;border:0px;outline:0px;vertical-align:=
baseline;background:transparent;font-family:proxima-nova-zendesk,system-ui,=
-apple-system,BlinkMacSystemFont,&quot;Segoe UI&quot;,Roboto,Oxygen-Sans,Ub=
untu,Cantarell,&quot;Helvetica Neue&quot;,Arial,sans-serif;font-weight:600"=
><font size=3D"2">CONFIDENTIALITY NOTICE: This email may contain confidenti=
al and privileged material for the sole use of the intended recipient(s). A=
ny review, use, distribution or disclosure by others is strictly prohibited=
.=C2=A0 If you have received this communication in error, please notify the=
 sender immediately by e-mail and delete the message and any file attachmen=
ts from your computer. Thank you.</font></span></i></blockquote></div></div=
>
</blockquote></div></div>

--0000000000002fc2aa065049c9a2--

