Test app: http://github.com/bobtfish/catalyst-app-bug-go_chain/tree/master
- - Bricas' Exception blog post
-
- http://bricas.vox.com/library/post/catalyst-exceptionclass.html
-
- Broken by recent exception refactoring
-
# Compatibility warnings to add:
- $self->config should warn as config should only ever be called as a
- class method.
+ class method (TESTS).
# Proposed functionality / feature additions:
See also: Catalyst::Plugin::Log::Dispatch and
http://github.com/willert/catalyst-plugin-log4perl-simple/tree
-# REFACTORING
+## Capture arguments that the plack engine component was run with somewhere,
+ to more easily support custom args from scripts (e.g. Gitalist's
+ --git_dir)
+
+## throw away the restarter and allow using the restarters Plack provides
+
+## remove per-request state from the engine instance
-## The horrible hack for plugin setup - replacing it:
+## be smarter about how we use PSGI - not every response needs to be delayed
+ and streaming
+
+# The horrible hack for plugin setup - replacing it:
* Have a look at the Devel::REPL BEFORE_PLUGIN stuff
I wonder if what we need is that combined with plugins-as-roles
-## App / ctx split:
+ * Catalyst::Engine::HTTP::Prefork no longer works since it requires
+ Catalyst::Engine::CGI which no longer is in the cataplack distribution.
+ Investigation shows moving CE:CGI to CE:HTTP:Prefork allows tests to pass.
+
+# PSGI
+
+## To do at release time
+
+ - Release psgi branch of Catalyst-Devel
+ - Release new Task::Catalyst
+ - Release 5.9 branch of Catalyst-Manual
+ - Release Catalyst::Engine::HTTP::Prefork with deprecation notice
+ (and maybe compat?)
+
+## Blockers
+
+ * Better docs for stopping using Engine::HTTP::Prefork and
+ starting using Starman, maybe?
+
+ * Test::WWW::Mechanize::Catalyst new release
+
+ * Test nginx middleware to determine if it is needed with:
+
+ root app - with use_request_uri_for_path
+ root app - without use_request_uri_for_path
+ non-root app - with use_request_uri_for_path
+ non-root app - without use_request_uri_for_path
+
+ If it isn't needed, remove. If it is needed, split it out into it's own
+ file and document why it's needed.
+
+ * I've noticed a small difference with Catalyst::Test. The latest stable
+ version include two headers, 'host' and 'https'. They are missing from
+ this version - Pedro Melo on list
+ ^^ Cannot replicate this? Mailed back to ask for tests..
+
+# App / ctx split:
NOTE - these are notes that t0m thought up after doing back compat for
- _component_class, may be inaccurate, wrong or missing things
+ catalyst_component_class, may be inaccurate, wrong or missing things
bug mst (at least) to correct before trying more than the first 2
steps. Please knock yourself out on the first two however :)
- Profit! (Things like changing the complete app config per vhost, i.e.
writing a config loader / app class role which dispatches per vhost to
differently configured apps is piss easy)
-