[Openid-specs-ab] RP Testing: regressions since december, service availability

[Roland Hedberg]

Good catch Filip!

I will change the code to allow for /joe :-)

> 28 mars 2017 kl. 22:59 skrev Filip Skokan <panva.ip at gmail.com>:
> 
> rp-discovery-webfinger-url fails because it does not account for  <https://github.com/rohe/oidctest/blob/793f7dcd5dbf89c22865b949b38fb0d0da710ce6/src/oidctest/cp/op.py#L100>URL syntax with i.e. /joe at the end of the subject <https://github.com/rohe/oidctest/blob/793f7dcd5dbf89c22865b949b38fb0d0da710ce6/src/oidctest/cp/op.py#L100>. 
> 
> parse_resource returns array with three members which is too much for only op_id and test_id unpack, resulting in ValueError: too many values to unpack and then the aforementioned 400. 
> 
> To confirm I changed my test to just submit https://rp.certification.openid.net:8080/node-openid-client/rp-discovery-webfinger-url <https://rp.certification.openid.net:8080/node-openid-client/rp-discovery-webfinger-url> as resource value, the test passes, but it should actually be https://rp.certification.openid.net:8080/node-openid-client/rp-discovery-webfinger-url <https://rp.certification.openid.net:8080/node-openid-client/rp-discovery-webfinger-url>/joe and maybe it should even account for a trailing '/', therefore parse_resource returning 4 members.
> 
> Hope it helps.

I most certainly does

>  rp-userinfo-bearer-body is still a mystery to me...

I think I know what’s amiss here.

— Roland
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.openid.net/pipermail/openid-specs-ab/attachments/20170329/110fa50b/attachment.html>