Skip Menu |

Preferred bug tracker

Please visit the preferred bug tracker to report your issue.

This queue is for tickets about the Plack-Middleware-MethodOverride CPAN distribution.

Report information
The Basics
Id: 102151
Status: rejected
Priority: 0/
Queue: Plack-Middleware-MethodOverride
People
Owner: Nobody in particular
Requestors: zefram [...] fysh.org
Cc:
AdminCc:
Bug Information
Severity: (no value)
Broken in: (no value)
Fixed in: (no value)

History Show all quoted text
  Mon Feb 16 07:42:05 2015 zefram [...] fysh.org - Ticket created
Subject: URI parsing change broke Catalyst-Runtime
Date: Mon, 16 Feb 2015 12:41:53 +0000
To: bug-Plack-Middleware-MethodOverride [...] rt.cpan.org
From: Zefram <zefram [...] fysh.org>
The changed URI parsing in Plack-Middleware-MethodOverride-0.12 is causing two test failures in Catalyst-Runtime. It was reported against Catalyst-Runtime as [rt.cpan.org #102129], but it looks like what C-R is testing is reasonable and that the change is therefore faulty. Attached patch restores the former URI parsing. -zefram

Message body is not shown because sender requested not to inline it.

  Mon Feb 16 09:44:50 2015 ARISTOTLE [...] cpan.org - Correspondence added
That is a bug in Catalyst, not in Plack::Middleware::MethodOverride. Any middleware using Plack::Request to parse query parameters ahead of Catalyst would expose the same problem, MethodOverride just happened to be the one that exposed the bug. I’ve already submitted a patch to fix Catalyst, c.f. https://github.com/perl-catalyst/catalyst-runtime/pull/81
  Mon Feb 16 09:44:50 2015 The RT System itself - Status changed from 'new' to 'open'
  Mon Feb 16 09:44:51 2015 ARISTOTLE [...] cpan.org - Status changed from 'open' to 'rejected'