[Openid-specs-fapi] [EXTERNAL] Re: Duplicate kids in jwks

Wed Jan 22 04:46:53 UTC 2020

I’ve closed the bitbucket Connect issue, since this is now being tracked in the conformance suite tracker.

                                                       -- Mike

From: Brian Campbell <bcampbell at pingidentity.com>
Sent: Friday, January 17, 2020 2:45 PM
To: Financial API Working Group List <openid-specs-fapi at lists.openid.net>
Cc: Mike Jones <Michael.Jones at microsoft.com>
Subject: Re: [Openid-specs-fapi] [EXTERNAL] Re: Duplicate kids in jwks

That the ticket https://bitbucket.org/openid/connect/issues/1127<https://nam06.safelinks.protection.outlook.com/?url=https%3A%2F%2Fbitbucket.org%2Fopenid%2Fconnect%2Fissues%2F1127&data=02%7C01%7CMichael.Jones%40microsoft.com%7C16683a94a2354995843c08d79b9ef9f0%7C72f988bf86f141af91ab2d7cd011db47%7C1%7C0%7C637148979409326744&sdata=0pZKwGWC6u3oZEQIExCVAZ3KrU5BhK5au90Eq%2FEdfa0%3D&reserved=0> indicates a Component of Core and Milestone of Errata suggests that an erratum change to specification(s) is intended. So that should be changed.

On Fri, Jan 17, 2020 at 3:02 PM Mike Jones via Openid-specs-fapi <openid-specs-fapi at lists.openid.net<mailto:openid-specs-fapi at lists.openid.net>> wrote:
Filip is correct.  The intent is only to enforce this behavior in certified implementations – not to change the specifications.  This is being done to increase interoperability.

There’s lots of cases where we intentionally constrain certified implementations beyond what’s strictly required by the specifications to increase interoperability.  For instance, the test “(OP-IDToken-kid) IDToken has kid [Basic, Implicit, Hybrid]” requires that the ID Token always contains a Key ID, even though the Connect spec only requires it when there are multiple keys in the JWK Set.  This was done so that it could be counted on.

Likewise, the WG decision to require unique “kid” values goes beyond the spec, but eliminates a number of potential errors resulting from odd corner cases.  I firmly believe that the working made the right call to enforce this in the certification suite.

                                                          -- Mike

From: Openid-specs-fapi <openid-specs-fapi-bounces at lists.openid.net<mailto:openid-specs-fapi-bounces at lists.openid.net>> On Behalf Of Filip Skokan via Openid-specs-fapi
Sent: Friday, January 17, 2020 1:42 PM
To: Financial API Working Group List <openid-specs-fapi at lists.openid.net<mailto:openid-specs-fapi at lists.openid.net>>
Cc: Filip Skokan <panva.ip at gmail.com<mailto:panva.ip at gmail.com>>
Subject: [EXTERNAL] Re: [Openid-specs-fapi] Duplicate kids in jwks

I don’t think it’s clear whether the intent was an errata or a proposal to the WG to further constrain the Connect certification profile.

I believe we could only do the latter.
Odesláno z iPhonu

17. 1. 2020 v 22:05, Brian Campbell via Openid-specs-fapi <openid-specs-fapi at lists.openid.net<mailto:openid-specs-fapi at lists.openid.net>>:

For better or worse JOSE allowed for duplicate kids and Connect didn't constrain it. So requiring uniqueness is a breaking change that an erratum shouldn't be doing. I guess that counts as a negative opinion towards the decision in https://bitbucket.org/openid/connect/issues/1127<https://nam06.safelinks.protection.outlook.com/?url=https%3A%2F%2Fbitbucket.org%2Fopenid%2Fconnect%2Fissues%2F1127&data=02%7C01%7CMichael.Jones%40microsoft.com%7C16683a94a2354995843c08d79b9ef9f0%7C72f988bf86f141af91ab2d7cd011db47%7C1%7C0%7C637148979409326744&sdata=0pZKwGWC6u3oZEQIExCVAZ3KrU5BhK5au90Eq%2FEdfa0%3D&reserved=0>

On Fri, Jan 17, 2020 at 12:50 AM Joseph Heenan via Openid-specs-fapi <openid-specs-fapi at lists.openid.net<mailto:openid-specs-fapi at lists.openid.net>> wrote:
Hi all,

I wanted to direct the FAPI working group to this discussion within the Connect working group:

https://bitbucket.org/openid/connect/issues/1127<https://nam06.safelinks.protection.outlook.com/?url=https%3A%2F%2Fbitbucket.org%2Fopenid%2Fconnect%2Fissues%2F1127&data=02%7C01%7CMichael.Jones%40microsoft.com%7C16683a94a2354995843c08d79b9ef9f0%7C72f988bf86f141af91ab2d7cd011db47%7C1%7C0%7C637148979409336743&sdata=yxXp3%2F3ljzQd8nGySVQhn2gpQ%2B%2FfiJK%2FO5p%2F%2F88WHsQ%3D&reserved=0>

Namely that duplicate kids are not permitted in JWKS.

A test for this was recently added to all the conformance tests, which caused one of the UK banks to opine:

it is valid for the JWK endpoint to return multiple KID instances, one for each ‘alg’ supported?
The spec calls for the alg PS256 or longer to be supported, so we also have (for instance) PS384, PS512. And although we may show a couple that we don’t need, my point is that it must be valid to show multiple key entries to support multiple valid alg values.

To some extent this seems a reasonable point, reusing a key across across two algs that can use the same key seems ok, and arguably perhaps better than having the key once without an ‘alg’ specified.

As this affects the FAPI certification tests, I wanted to check the FAPI WG agrees with the decision in https://bitbucket.org/openid/connect/issues/1127<https://nam06.safelinks.protection.outlook.com/?url=https%3A%2F%2Fbitbucket.org%2Fopenid%2Fconnect%2Fissues%2F1127&data=02%7C01%7CMichael.Jones%40microsoft.com%7C16683a94a2354995843c08d79b9ef9f0%7C72f988bf86f141af91ab2d7cd011db47%7C1%7C0%7C637148979409336743&sdata=yxXp3%2F3ljzQd8nGySVQhn2gpQ%2B%2FfiJK%2FO5p%2F%2F88WHsQ%3D&reserved=0> - any opinions (positive & negative) would be great please.

Thanks

Joseph Heenan
OpenID Certification Team

_______________________________________________
Openid-specs-fapi mailing list
Openid-specs-fapi at lists.openid.net<mailto:Openid-specs-fapi at lists.openid.net>
http://lists.openid.net/mailman/listinfo/openid-specs-fapi<https://nam06.safelinks.protection.outlook.com/?url=http%3A%2F%2Flists.openid.net%2Fmailman%2Flistinfo%2Fopenid-specs-fapi&data=02%7C01%7CMichael.Jones%40microsoft.com%7C16683a94a2354995843c08d79b9ef9f0%7C72f988bf86f141af91ab2d7cd011db47%7C1%7C0%7C637148979409346737&sdata=GdG4VRtmHA1%2BdhFLj7te8M9Qw4Joe%2BrXijtLiqKNRYw%3D&reserved=0>

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 sender immediately by e-mail and delete the message and any file attachments from your computer. Thank you._______________________________________________
Openid-specs-fapi mailing list
Openid-specs-fapi at lists.openid.net<mailto:Openid-specs-fapi at lists.openid.net>
http://lists.openid.net/mailman/listinfo/openid-specs-fapi<https://nam06.safelinks.protection.outlook.com/?url=http%3A%2F%2Flists.openid.net%2Fmailman%2Flistinfo%2Fopenid-specs-fapi&data=02%7C01%7CMichael.Jones%40microsoft.com%7C16683a94a2354995843c08d79b9ef9f0%7C72f988bf86f141af91ab2d7cd011db47%7C1%7C0%7C637148979409346737&sdata=GdG4VRtmHA1%2BdhFLj7te8M9Qw4Joe%2BrXijtLiqKNRYw%3D&reserved=0>
_______________________________________________
Openid-specs-fapi mailing list
Openid-specs-fapi at lists.openid.net<mailto:Openid-specs-fapi at lists.openid.net>
http://lists.openid.net/mailman/listinfo/openid-specs-fapi<https://nam06.safelinks.protection.outlook.com/?url=http%3A%2F%2Flists.openid.net%2Fmailman%2Flistinfo%2Fopenid-specs-fapi&data=02%7C01%7CMichael.Jones%40microsoft.com%7C16683a94a2354995843c08d79b9ef9f0%7C72f988bf86f141af91ab2d7cd011db47%7C1%7C0%7C637148979409356727&sdata=A%2BdnB6KNuNEgW3ygDGv5NGX424PMIaIBeaoU9oa2S3M%3D&reserved=0>

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 sender immediately by e-mail and delete the message and any file attachments from your computer. Thank you.
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.openid.net/pipermail/openid-specs-fapi/attachments/20200122/09702c04/attachment-0001.html>