Skip Menu |

This queue is for tickets about the Catalyst-Authentication-Credential-OpenID CPAN distribution.

Report information
The Basics
Id: 56330
Status: resolved
Priority: 0/
Queue: Catalyst-Authentication-Credential-OpenID
People
Owner: Nobody in particular
Requestors: TREVELYAN [...] cpan.org
Cc:
AdminCc:
Bug Information
Severity: Critical
Broken in: (no value)
Fixed in: (no value)

History Show all quoted text
  Tue Apr 06 15:57:00 2010 TREVELYAN [...] cpan.org - Ticket created
Subject: Callculate $current url incorrectly
Hi, This line in OpenID.pm: my $current = $c->uri_for($c->req->uri->path); # clear query/fragment... should be: my $current = $c->uri_for('/'.$c->req->path); # clear query/fragment... otherwise it will get the wrong url if myapp is not at the domain root, and/or the login action is not in Root.pm Regards, Thorben
  Tue Apr 06 15:58:00 2010 TREVELYAN [...] cpan.org - Severity Critical added
  Tue Apr 06 23:08:39 2010 ashley [...] cpan.org - Correspondence added
From: ashley [...] cpan.org
Thanks! I will get this and see about the extensions stuff as soon as possible. On Tue Apr 06 15:57:00 2010, TREVELYAN wrote: Show quoted text
> Hi, > > This line in OpenID.pm: > > my $current = $c->uri_for($c->req->uri->path); # clear > query/fragment... > > > should be: > > my $current = $c->uri_for('/'.$c->req->path); # clear > query/fragment... > > otherwise it will get the wrong url if myapp is not at the domain root, > and/or the login action is not in Root.pm > > Regards, > > Thorben
  Tue Apr 06 23:08:40 2010 The RT System itself - Status changed from 'new' to 'open'
  Sat Jan 25 17:50:52 2014 TREVELYAN [...] cpan.org - Status changed from 'open' to 'resolved'