TWMC is fixed

[catagits/Catalyst-Runtime.git] / TODO

diff --git a/TODO b/TODO
index c131e71..ebfc646 100644 (file)
--- a/TODO
+++ b/TODO
@@ -24,51 +24,57 @@ subclass of Catalyst::Log, no ::Plugin:: needed.
 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)
 
-##  PSGI
+## throw away the restarter and allow using the restarters Plack provides
 
-###  Blockers
+## remove per-request state from the engine instance
 
-  * Test all the options work on all of the scripts
-  * Fix nginx middlewares so that they are generic, or can somehow
-    be used by people with their own .psgi files
-  * Fix a sane / nicer way to do custom engines.
+## be smarter about how we use PSGI - not every response needs to be delayed
+    and streaming
 
-#### Script survey
+#  The horrible hack for plugin setup - replacing it:
 
-##### myapp_web_fastcgi.pl
+ * Have a look at the Devel::REPL BEFORE_PLUGIN stuff
+   I wonder if what we need is that combined with plugins-as-roles
 
-Looks to me like we are mapping --deamon to --detach but I think the modern Plack FCGI handler prefers --deamonize
+  * 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.
 
-Although --pidfile is supported --pid seems to be preferred, and if we are bothering to map, why not map for the future?
+#  PSGI
 
-##### myapp_web_server.pl
+##  To do at release time
 
---fork, this gets passed and Plack doesn’t complain, but it doesn’t fork.  Maybe we could just detect this switch and complain about it (say you should use plackup and Starman, for example?)
+  - 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?)
 
---keepalive, passed, no complaint but doesn’t really seem to do anything.
+##  Blockers
 
---pidfile, --background, these also seem to do nothing.
+  * Better docs for stopping using Engine::HTTP::Prefork and
+    starting using Starman, maybe?
 
-###  Nice to have
+  * Test nginx middleware to determine if it is needed with:
 
-  * <@rafl> i've been thinking of maybe providing
-    MyApp->apply_default_middlewares($psgi_app)
-  * 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
-  * be smarter about how we use PSGI - not every response needs to be delayed
-    and streaming
+    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
 
-##  The horrible hack for plugin setup - replacing it:
+    If it isn't needed, remove. If it is needed, split it out into it's own
+    file and document why it's needed.
 
- * Have a look at the Devel::REPL BEFORE_PLUGIN stuff
-   I wonder if what we need is that combined with plugins-as-roles
+  * 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:
+# App / ctx split:
 
   NOTE - these are notes that t0m thought up after doing back compat for
          catalyst_component_class, may be inaccurate, wrong or missing things