[Openid-specs-ab] Comments on the OpenID Connect Standard spec 1.0 draft 4
ejay at mgi1.com
Tue Sep 20 18:19:22 UTC 2011
The query parameters need to be sent even when "request" parameter is sent
because the request needs to conform to OAuth specs.
The "request" parameter is an extension parameter used for creating more complex
requests and as a way to sign/encrypt the request. Therefore the query
parameters need to be present in the "request" object also and will take
From: Roland Hedberg <roland.hedberg at adm.umu.se>
To: George Fletcher <gffletch at aol.com>
Cc: "openid-specs-ab at lists.openid.net" <openid-specs-ab at lists.openid.net>
Sent: Mon, September 19, 2011 11:49:53 PM
Subject: Re: [Openid-specs-ab] Comments on the OpenID Connect Standard spec 1.0
20 sep 2011 kl. 03:37 skrev George Fletcher:
> * Section 188.8.131.52
> The second paragraph says that parameters specified in the "OpenID Request
>Object" take precedence over query parameters. Yet the non-normative example,
>shows the same parameter in both the query string and the OpenID Request Object.
>Given that the Request Object takes precedence, isn't just the request object
>enough? So the last example in section 184.108.40.206 could be...
I went through the same reasoning but I came out the other end with the idea
that the parameters that matter, those you want to sign, they should be in the
request JWT and those that isn't vital (are there any such) could be in the
Anyway I also see no reason for parameters to be in both.
> * Section 220.127.116.11
> This probably isn't an issue, but ensuring the entire URL does not exceed 512
>bytes, requires both the AS and the Client to work together. If the client has a
>really large state value, and the AS has a large code value, the combined length
>could be greater than 512.
Agreed, a bad behaved client can make it impossible for a server to construct
URLs shorter then 512 bytes.
Openid-specs-ab mailing list
Openid-specs-ab at lists.openid.net
-------------- next part --------------
An HTML attachment was scrubbed...
More information about the Openid-specs-ab