minor spec update / new Net::OpenID::Consumer and ::Server modules

Brad Fitzpatrick brad at danga.com
Tue May 24 22:47:14 PDT 2005


On Wed, 25 May 2005, Martin Atkins wrote:

> I updated to 0.04 and modified my check_url call:
>      my $ret_url = $cid->check_url(
>          return_to => "http://goathack.livejournal.org:9016/openid",
>          trust_root => "http://goathack.livejournal.org:9016/openid",
>          delayed_return => 1,
>      );
>
> It looks like all of the URL arguments in the response are being
> separated by question marks rather than ampersands as they should be.
>
> In push_url_arg:
>
>      while (@_) {
>          my $key = shift;
>          my $value = shift;
>          $$uref .= $got_qmark ? "&" : ($got_qmark = 0, "?");
>          $$uref .= eurl($key) . "=" . eurl($value);
>      }
>
> I guess that fourth line is supposed to set $got_qmark to true, not false.

Doh.



More information about the yadis mailing list