Skip Menu |

This queue is for tickets about the Catalyst-Plugin-SubRequest CPAN distribution.

Report information
The Basics
Id: 78836
Status: resolved
Priority: 0/
Queue: Catalyst-Plugin-SubRequest

People
Owner: Nobody in particular
Requestors: nebulous [...] crashed.net
Cc:
AdminCc:

Bug Information
Severity: Normal
Broken in:
  • 0.17
  • 0.18
  • 0.19
Fixed in: 0.16



Subject: Multiple subrequests fail or are unpredictable
Multiple subrequests throw an error when run within single threaded server, and silently fail in multithreaded servers. I see edenc added the test from nopaste #204854 but it's important to note that particular test will pass when run from Catalyst::Test yet will fail when the same app is run within Catalyst::Script::Server. Works in versions prior to 0.17 *I reserve the right to be completely wrong in the way I'm thinking about this and/or what code is to blame. This is not medical advice. Ask your doctor before applying any patches, concepts, or tests.
On Tue Aug 07 10:00:52 2012, NEBULOUS wrote: Show quoted text
> Multiple subrequests throw an error when run within single threaded > server, and silently fail in multithreaded servers. > > I see edenc added the test from nopaste #204854 but it's important to > note that particular test will pass when run from Catalyst::Test yet will > fail when the same app is run within Catalyst::Script::Server. > > Works in versions prior to 0.17
This is a duplicate of #78821 and has been fixed in latest svn. Can you check out the repo and verify that the fix works for you?
On Tue Aug 07 10:00:52 2012, NEBULOUS wrote: Show quoted text
> Multiple subrequests throw an error when run within single threaded > server, and silently fail in multithreaded servers. > > I see edenc added the test from nopaste #204854 but it's important to > note that particular test will pass when run from Catalyst::Test yet will > fail when the same app is run within Catalyst::Script::Server.
Also, I've noticed that issue and have been trying to think of a decent way to test this, but couldn't quite get my brain wrapped around it yet, if you could contribute a failing test for that case, I'd appreciate it immensely.
Show quoted text
> This is a duplicate of #78821 and has been fixed in latest svn. Can you > check out the repo and verify that the fix works for you?
Confirmed. r14335 works for me.