Fix the fcgi script tests.

Merge branch 'master' into psgi

master:
Depend on n:c 0.12 to work on perl >= 5.11.2.
Version 5.80018.
canonical() is a no-op for the base uri.
Fix a deprecation warning in the tests.
Fix URI bug masked by HTTP::Request::AsCGI
Deprecate bare imports of Catalyst::Test - either use an app name or don't run the import method. As-per r12564
Apply patch to clarify uri_for action from Octavian Rasnita on list
Version 5.80017.
require autoclean once only
Bump version of ::Role::WithOverloading
Bump dep
Un stupid
Correctly pass argv option into Catalyst::Engine::HTTP
Changelog Adopt::NEXT warnings
Back out r12493, use \Q instead
Don't screw over people using --detach, <sigh>
Clarify comment.

Conflicts:
lib/Catalyst/Engine/CGI.pm
lib/Catalyst/Engine/HTTP.pm
t/aggregate/unit_core_script_server.t

Make build_psgi_app public and add MyApp->psgi_app.

Depend on n:c 0.12 to work on perl >= 5.11.2.

Depend on a ReverseProxy middleware that spits out secure envs for requests forwarded to secure ports.

Version 5.80018.

canonical() is a no-op for the base uri.

Fix a deprecation warning in the tests.

Do the right thing with FCGI args, the hostname and the keepalive options in the scripts so that the Plack servers get them correctly.

There are still a load of options for the dev server (like writing a PID file) that we'll need to add to the plack server (or do ourselves manually?), and the entire set of options we actually pass into the application via ->run is unused by it.

I'm not sure if we are safe to change the original option passing convention or not - need to look at alternate engines and stuff..

Move everything back out into the scriptrole, allow the scripts to force their engine..

Fix URI bug masked by HTTP::Request::AsCGI

Canonicalize the url becuse we're building it from the always-undecoded REQUEST_URI, not PATH_INFO.

Well, apparently this *is* necessary, just the extra_env stuff isn't :/

This reverts commit 87d4377bb66a6530cd2e13c4e770e7b1f0aaf7d3.

Delete unfixable FAIL.

It's unnecessary under PSGI anyway. The PSGI servers won't let get undecoded
paths through to us.

Port t/aggregate/live_engine_request_escaped_path.t to psgi.

Deprecate bare imports of Catalyst::Test - either use an app name or don't run the import method. As-per r12564

Skip Engine::CGI-specific tests until we find a better place for them.

At least pass the port param to _run_psgi_app to get http-server.t running again.

Stop expecting the Host header for proxy requests to be the original one.

Doing this was probably stupid in the first place, and now we're handing all
the frontend proxy stuff off to plack, we shouldn't even care anymore. The
middleware is free to set HTTP_HOST to HTTP_X_FORWARDED_HOST for us.

Start supporting running behind reverse proxies again.

Inline the OverrideEnv middleware into Catalyst::Test.

Apply patch to clarify uri_for action from Octavian Rasnita on list

Improve handling errors during prepare.

Look up the method just once.

Properly merge the extra env var into the psgi environment.

Don't dump psgi.input as it can't be deserialized.

Adapt to the changed _customize_request calling convention.

Properly set REMOTE_USER in the request env.

Add a way to specify additional psgi values in the psgi env via Catalyst::Test.

Version 5.80017.

Use Plack::Loader and push the running of plack back into the engine code. Unsure if here or scriptrole is the best place for it, thoughts? (48 => 13 fails though)

Delete code. 64 => 48 subtests failed

Remove outdated comment and some trailing whitespace.

Fix typo.

depend on Plack 0.903 for streaming interface support in Plack::Test

Require Plack..

require autoclean once only

Bump version of ::Role::WithOverloading

Clear up the response callback and the writer object after each request.

Don't pass $ctx down to read().

No, don't warn in the log. clients being retarded don't concern us usually.

Let read_chunk read from psgi.input.

There's no Content-Length for streaming responses.

Remove other engines.

Clear the request env after finishing a request.

Delay erroring out on wrong usage of Catalyst::Test.

We should probably deprecate

use Catalyst::Test;

(without an app name) in favour of

use Catalyst::Test ();

Tell the writer object when we're done.

Start porting Catalyst::Test to Plack::Test.

Work towards supporting psgi.

Bump dep

Un stupid

Correctly pass argv option into Catalyst::Engine::HTTP

Changelog Adopt::NEXT warnings

Back out r12493, use \Q instead

Don't screw over people using --detach, <sigh>

Clarify comment.

Stop supressing Adopt::NEXT warnings.

Clarify comment

Doc fix

Fix regex special characters screwing things up by not using regexes

Someone think of a less fugly way of doing this please? Fixes using rewrite rules to ask for a sub-path in your app with apache in some combinations..

Fix RT#52898, __MOP__ removal breaking debug screen with C::P::Session

Bug fix for issue reported on the mailing list by Toby Corkindale

moar better warnings fix.

Changelog, bump version

Remove warning when running tests aggregated

Unfuck that as well

Fix RT#52630

Additional test

More tests for the prepare_path thing, fix said tests, changelog

Changelog

And more tests and fixing for the same thing

Fixes port environment, RT#52604

Regression for 5.80015 when rewriting an app, nanonyme++, Khisanth++

Fix tests from r12252, add comments to make the behavior explicit

Doc --host change

Don't default to localhost for --host

Tweak docs for --host to say it accepts a name or IP

Win32 fix: reverting commit 12210 (after a discussion with t0m) - all tests pass on Win32/strawberry perl 5.8.9 and 5.10.1

Doc change for --restartdirectory

fix pod for Server script so options match actual attr name

Add restartdirectory as alias for restart_directory, for backcompat

Undocument $request->user.

Skip on win32, hopefully we'll get a better answer than this, but lets stop failing

Win32 fix, kmx++

I can't stand the over-long debug screens any more. Suggestions on how to do this in a less gross way?

add another failing test for Chained CaptureArgs preference

And that's what I get for not really applying the patch and doing it manually. Fail..

Pod nits, rt#52370

Changes, bump version

The documentation example had the variables the opposite way round to my working nginx config. CLEARLY this meant the documentation needed the variables (but not their values) transposing. Yes, yes - I really am _that_ dumb.

Fix bug in Catalyst::Engine which could cause it to all go wrong if read returned '0' as per thread on mailing list re 'Wrong Content-Length value' and clarify docs about read and read_chunk methods in Catalyst::Engine

Bump deps and changelog for another dev release with new scripts and scary path resolving changes so I can have another call to test.

TODO out gshanks chained tests

Rewrite changelog entry to be more complete

Merge 'trunk' into 'fix_path_info_decoding'

r12121@t0mlaptop (orig r12086):  t0m | 2009-11-30 22:59:15 +0000
Require new M::I

Require new HTTP::Request::AsCGI

Require new M::I

Back out r12083, also clarify the docs - like this works for me now, and the bottom para seems the wrong way round...?

I think the nginx hackery can just die with the new code in ::Engine::CGI

And yea, these test cases now pass