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

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..

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.

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.

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.

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..

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.

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

Commit miyagawa's fix for PATH_INFO decoding. I think that this also implies changes / fixes in the fcgi _fix_env stuff

Merge 'trunk' into 'better_scripts'

r13204@nc10 (orig r12073):  rafl | 2009-11-29 16:43:17 +0000
Depend on a recent leakchecker to stop everything from crapping over itself if PadWalker is too old.

Depend on a recent leakchecker to stop everything from crapping over itself if PadWalker is too old.

Merge 'trunk' into 'better_scripts'

r13191@nc10 (orig r12060):  gshank | 2009-11-27 22:43:11 +0000
testcase for Chained bug - dispatches to wrong action

Tidy things up a little and fix pod usage.

This is not a release.

Less trailing whitespace.

Remove useless lexical.

Port numbers need to be > 0.

Tidy up some attr definitions.

Properly validate restart regexen.

Make follow_symlinks more consistent with the other restarter args.

Depend on a new CMOP because we rely on its new load_class error messages.

Adding the files you need for the tests you've committed helps them pass, idiot :)

testcase for Chained bug - dispatches to wrong action

Bump version in runtime branch

Pass app name into constructor

Merge 'trunk' into 'better_scripts'

r12057@t0mlaptop (orig r12022):  t0m | 2009-11-26 01:21:22 +0000
Back out 11979 until I work out why IPC::Run is getting hold of the FCGI file handle
r12070@t0mlaptop (orig r12035):  t0m | 2009-11-26 21:29:32 +0000
Fix test on bleadperl RT#52100

Fix test on bleadperl RT#52100

Fix typo, bricas++, add simple test for restarter arguments