Hans Hübner <hans.huebner@gmail.com> writes:
the patch looks good. Can you please provide a patch with a docstring and a documentation update?
Sure, I'm attaching a patch. I tweaked the docstring for process-request also. This patch is simple in the sense that it provides an extra configuration point without otherwise interfering with the API. BTW, having now looked into using process-request as a "configuration point", I would suggest that there should be some refactoring: Move the pieces of it that are intrinsic to hunchentoot, such as bindings etc. for *tmp-files*, *headers-sent*, *request*, handler-done, and *within-request-p* [why not just (not (null *request*))?] into process-connection (or some internal defun), and keep just the stuff that implements the default condition handling. The point would be to have two well-defined configuration points: process-request that does condition handinling, and dispatch-request that is within the scope of said condition handling. -- Frode V. Fjeld Netfonds Bank ASA