added missing dep

Fix the errors in finalize_headers when running on Engine::PSGI

Replace the setup_engine workaround with a better one

Instead of letting the .psgi setup_engine('PSGI') and then trying
to undo the damage, add a flag that makes setup_engine not really
load the requested engine if we're in the middle of loading a myapp.psgi

now better support for legacy psgi (Catalyst::Engine::PSGI) running with plackup under cataplack

Write out the pid file after double fork.

Therefore we get the pid of the actual server process saved, rather than
that of the runner

Fix background option

Hacking the server script

Add the crux of options that need fixing from jnap's survey

Merge branch 'master' into psgi

* master:
  More disable_component_resolution_regex_fallback
  give an example for $c->request->path
  fix typo

Conflicts:
Changes

More disable_component_resolution_regex_fallback

give an example for $c->request->path

Signed-off-by: Caleb Cushing <xenoterracide@gmail.com>

fix typo

Merge branch 'master' into psgi

* master:
  Unbreak remote server tests
  Add MYMETA.yml to .gitignore
  Add missing Catalyst::Devel author dep
  Append $\ in Catalyst::Response->print

Unbreak remote server tests

Add MYMETA.yml to .gitignore

Add missing Catalyst::Devel author dep

Append $\ in Catalyst::Response->print

This more closely matches IO::Handle's behaviour.

Unbreak remote server tests

Merge branch 'master' into psgi

* master:
  fix some typos
  Fixed situation where a detach($action) from a forward within auto was not breaking out correctly
  Add a test to make sure that a die in a Controller's end sub is propogated as a 500 error
  Unbreak the pod-coverage tests
  Yes, it is
  Remove special-casing of '*' for get_action_methods
  Pull a bunch of gross hacks to get actions => { '*' => ... } working again
  Add a test for configuring attributes for all actions
  Add a test view for dumping actions
  Make this error message more readable
  Make this error message more readable
  Agreed

fix some typos

removed backcompat code for old PSGI Engine since this would be handled somewhere else better

got all the tests running again

Merge branch 'psgi' into origin/psgi

Fixed situation where a detach($action) from a forward within auto was not breaking out correctly

Document differences in Catalyst::Test as found by autarch++

Pass extra args from ScriptRunner->run to ->new_with_options.

This is to allow you to reuse the loading logic in ::ScriptRunning, without
being forced to push things into %ENV to get them picked up

Make HTTP force standalone

public apply_default_middlewares method

first attempt at a patch to enable compatibility with psgi files based on Catalyst::Engine::PSGI and also a change to stop the author side pod coverage tests complaining pointlessly

Add a test to make sure that a die in a Controller's end sub is propogated as a 500 error

Unbreak the pod-coverage tests

Without removing those imports, it expects us to document them.

Yes, it is

Remove special-casing of '*' for get_action_methods

Instead, pull '*' out of _controller_actions early on. The part where they're
applied to actions is still gross.

Pull a bunch of gross hacks to get actions => { '*' => ... } working again

Add a test for configuring attributes for all actions

Add a test view for dumping actions

Make this error message more readable

Make this error message more readable

Agreed

Merge branch 'master' into psgi

* master:
  A failing reverse lookup doesn't imply it's a local address
  Tiny tidyups to copyright and license

Conflicts:
lib/Catalyst/Engine/HTTP.pm

A failing reverse lookup doesn't imply it's a local address

Fix a syntax error introduced in d367082

Use the Plack LighttpdScriptNameFix middleware.

This replaces our home-baked solution with something which works better
and in more cases

Update TODO

Clean up FIXMEs

Fix a warning under aggregation

Pass script options into auto loader

Plack::Handler::XXX therefore gets the options even when auto-detecting.

Retrieve args for old scripts so FCGI --listen works

Force FCGI detection for FastCGI scripts.

Do not commit conflicts, idiot

Fix bug stopping fastcgi script working

Update todo

Update todo

Fix the breakage the last commit introduced

make ->setup_engine($engine) supported API again

It should always take preference over CATALYST_ENGINE. The compat hack is really
just PSGI specific.

No, we don't want this

If we'd check it, we'd have to check it on every request. That's entirely not
worth it as I'm not even aware of any server that doesn't support psgi.streaming
right now, let alone a server people actually use.

If people complain, documentation suggesting to load the BufferedWrite
middleware needs to be written.

Document middlewares applied automatically

Explicitly document the wrong Engine::PSGI thing.

I did it accidentally, and it took rafl half an hour to chase down in d5024dd882,
so someone as dumb as me is likely to do it (just substituting the method name,
rather than removing the additional closure) when upgrading from ::Engine::PSGI.

Update TODO

Partially unify local and remote request codepaths

Use Plack::Test::ExternalServer in Catalyst::Test remote requests

We already have a warning for this

Fix the test psgi file

psgi_app is just a getter

Don't assume the app gets loaded before the first request

First stab documenting writing a .psgi file

Legacy .psgi files are never loaded

Ignore old-style .psgi files

De-uglify a comment

Document status of the misc engines.

Both SCGI and XMPP appear to be bork

Add a test showing how having an Engine::PSGI .psgi file breaks.

Notably, this .psgi file is in the app root, whereas ::Engine::PSGI's docs and helper suggest that
you create it in script/. So I don't have a problem with not supporting the .psgi working for tests
until you've converted it if you move it into the app root really - however it would be nice to be
able to detect it was broken, rather than just hanging.

Test showing how having a .psgi file will ruin your day

local_request is now tested

(and deprecated!)

Add tests for the deprecated local_request

strictures and warnings for live_catalyst_test.t

This was fixed already

Fix the conditional here.

We do always need a trailing slash if we have a server path and either
just a /, or no uri at all (e.g. http://localhost). This matters when
we're mounted at a non-root path, as cataltst needs to see
http://localhost/non_root_path/, rather than http://localhost/non_root_path,
otherwise we'll try to resolve /non_root path as an action.

Trim hatespace

Tiny tidyups to copyright and license

The IIS 6 script name fix is now part of Plack

Depend on it and use it.

Merge branch 'master' into psgi

* master:
  Fix Path actions debug screen to display number of arguments

Fix Path actions debug screen to display number of arguments

Merge branch 'master' into psgi

* master:
  Make default test names reflect reality more and fix a buglet, rafl++
  Give action_ok, action_redirect and action_notfound default test names

Make default test names reflect reality more and fix a buglet, rafl++

Give action_ok, action_redirect and action_notfound default test names

Merge branch 'master' into psgi

* master:
  Avoid super long command lines when not aggregating
  Improve the documentation for various functions in Catalyst::Test

Avoid super long command lines when not aggregating

This probably also fixes prove -lr t/, which people have been complaining about
every now and then.

Improve the documentation for various functions in Catalyst::Test

Deprecate local_request and remote_request

Move out building of the horrible ctx_request closure

Also move out the rest, while we're at it. This is intended to aid a subsequent
refactoring.

Remove some kane-isms that hurt my eyes

Run http-server.t tests on something other than the server root

Remove testsuite's assumptions about where it's mounted on a web server

Make http-server.t use Test::TCP

No dep, as it's author-only and pulled in by Plack already anyway.

Don't use the server.pl script in the live tests

Instead, just spin up a server with Plack::Loader. The script classes are
already tested separately, so we only need this test to make sure live tests
work.

This is intended to aid a subsequent refactoring.

.psgi stuff is now sorted out

::Wx is declared dead

lighttpd and iis6 now work

Reduce code repetition

Add legacy workaround for nginx PATH_INFO