Skip Menu |

This queue is for tickets about the Net-OpenID-Consumer CPAN distribution.

Report information
The Basics
Id: 47349
Status: resolved
Priority: 0/
Queue: Net-OpenID-Consumer

People
Owner: Nobody in particular
Requestors: cnandor [...] cpan.org
Cc:
AdminCc:

Bug Information
Severity: Important
Broken in: 1.03
Fixed in: 1.12



In Net::OpenID::URIFetch is this code: if (my $ts = ($ref->{Headers}->{'last-modified'})) { $req->if_modified_since($ts); } If I am not mistaken, last-modified is a string, and if_modified_since expects numeric. I believe instead of asking for the Last-Modified header, youw ant to call $res->last_modified directly, which gives you the value. Or HTTP::Date::str2time($ts). Which is how I am patching my local copies, so I don't continue to get errors like: Argument "Thu, 18 Jun 2009 16:55:05 GMT" isn't numeric in gmtime at /usr/local/lib/perl5/site_perl/5.10.0/HTTP/Date.pm line 36. Cheers! -- Chris Nandor pudge@pobox.com http://pudge.net/ Slashdot / SourceForge pudge@slashdot.org http://slashdot.org/
Looks like this bug was introduced in the modifications after we forked URI::Fetch.
I believe this is fixed in Net-OpenID-Consumer 1.12 If you wish to try it out please make sure you've also installed the latest version of Net-OpenID-Common. Feel free to re-open (or start a new ticket) if it turns out I'm mistaken about this. Thanks for the report and sorry this took so long to get to... -- Roger Crew (new co-maintainer as of a few weeks ago)