Hi Vittorio! Thanks for the answers. I'm ok with the motivation for using "application/at+jwt", thanks for clarifying, and thanks for addressing 2. Re 1. IMO it would clarify the intention of the SHOULD to add what you wrote in response to my comment into the draft itself, but I'll leave it up to you.
Francesca On 14/04/2021, 09:19, "Vittorio Bertocci" <vittorio.berto...@auth0.com> wrote: Hi Francesca, Thanks for your review and thoughtful comments! Comments below. > 1. ----- > [...] While it is reasonable to expect that a RS receiving an unencrypted token after requesting it to be encrypted will reject it, there are a number of cases where the RS might elect to do otherwise. For example, the solution might be already working in production: the encryption requirement might be an improvement that is still propagating thru the system, and there might still be access tokens cached in clients that the RS might still be willing to accept to guarantee business continuity. "SHOULD" gives a strong signal to implementers of what the desired behavior is, but leaves them some freedom to accommodate situations like the aforementioned one. > 2. --- > [...] Fair enough. I added the following text at the beginning of 2.1. Thanks for catching this. JWT access tokens MUST be signed. Although JWT access tokens can use any signing algorithm[..] This change will appear in the next revision, which I will post soon. > 3. ----- > [...] Formally, I agree that JOSE would also work. The choice of media type derives from https://tools.ietf.org/html/rfc7519#section-10.3.1. There is no functional difference between JWS and JWE in the intent a client has when calling an RS, here there's not much to be gained in using different MIME types for those cases. Furthermore, whereas developers are familiar with the term "JWT", both from direct use and thanks to the popularity of OpenID Connect (which does use application/jwt), terms like JWS, JWE or JOSE wouldn't be as promptly understood as JWT. Throughout the discussions in the last couple of years, the consensus on the use of at+jwt was solid- my hope is that will make intuitive sense for implementers, too. On 4/4/21, 11:01, "Francesca Palombini via Datatracker" <nore...@ietf.org> wrote: Francesca Palombini has entered the following ballot position for draft-ietf-oauth-access-token-jwt-12: No Objection When responding, please keep the subject line intact and reply to all email addresses included in the To and CC lines. (Feel free to cut this introductory paragraph, however.) Please refer to https://www.ietf.org/iesg/statement/discuss-criteria.html for more information about IESG DISCUSS and COMMENT positions. The document, along with other ballot positions, can be found here: https://datatracker.ietf.org/doc/draft-ietf-oauth-access-token-jwt/ ---------------------------------------------------------------------- COMMENT: ---------------------------------------------------------------------- Thank you for the work on this document. Please find some comments and clarifying questions below. Francesca 1. ----- registration. If encryption was negotiated with the authorization server at registration time and the incoming JWT access token is not encrypted, the resource server SHOULD reject it. FP: Why is this just SHOULD and not MUST? In which case does it make sense to accept a non-encrypted token when encryption was negotiated? 2. ----- Section 2.1: Section 4). JWT access tokens MUST NOT use "none" as the signing algorithm. See Section 4 for more details. Section 4: For the purpose of facilitating validation data retrieval, it is here RECOMMENDED that authorization servers sign JWT access tokens with an asymmetric algorithm. ... o The resource server MUST validate the signature of all incoming JWT access tokens according to [RFC7515] using the algorithm specified in the JWT alg Header Parameter. The resource server FP: It might be obvious, but I think it would be useful to have an explicit sentence stating that JWT MUST be signed. The quoted text from Section 2.1 seem to imply it. Section 4 only RECOMMENDS that the JWT is signed with and asymmetric algorithm. Later on, Section 4 implies that all JWT are signed. On the other hand I note that encryption can be negotiated (and is optional) from the followig point; in that case it is not clear that the token is still signed (so the nested JWT would be a JWE nested in a JWS), or only JWE is used. What I am looking for is simple clarifications to be added for example in the introduction. o If the JWT access token is encrypted, decrypt it using the keys and algorithms that the resource server specified during registration. If encryption was negotiated with the authorization 3. ----- On the same note, and depending on the previous answer, why is the media type registered and used "application/at+jwt" and not something like "application/at+jws"/"application/at+jwe" or rather "application/at+jose" to be compliant with https://protect2.fireeye.com/v1/url?k=cf968f69-900db66b-cf96cff2-869a14f4b08c-d9afc9515caac1c4&q=1&e=e0795065-865a-4950-87a5-ac1b2b1d8e58&u=https%3A%2F%2Fwww.rfc-editor.org%2Frfc%2Frfc7515.html%23section-9.2.1 ? I think that the structure transported is in fact a JWS or a JWE, rather than the JWT, and if that's the case that should be made clear in the text (one example where this could be clarified is in the following sentence) Resource servers receiving a JWT access token MUST validate it in the following manner. _______________________________________________ OAuth mailing list OAuth@ietf.org https://www.ietf.org/mailman/listinfo/oauth