Revision: 14445
Author: adrian.chadd
Date: Wed Mar  3 18:45:18 2010
Log: More braindumping.


http://code.google.com/p/lusca-cache/source/detail?r=14445

Modified:
 /branches/LUSCA_HEAD/src/client_side.c

=======================================
--- /branches/LUSCA_HEAD/src/client_side.c      Wed Mar  3 18:24:33 2010
+++ /branches/LUSCA_HEAD/src/client_side.c      Wed Mar  3 18:45:18 2010
@@ -2223,6 +2223,14 @@
      * If http->entry != NULL then there's an existing object to piggy back
      * onto; so the store client registration occurs and the object is
      * copied in via storeClientCopyHeaders().
+     *
+     * .. TRACE handling isn't entirely clear either. For a non-terminal
+ * TRACE request, it should just bump it upstream or toss out a forwarding
+     * error. It doesn't bother trying a cache lookup; it just processes
+ * it as a MISS. But there's no explicit setting of http->entry to NULL; + * I believe it simply expects the only entry point to have it NULL (ie,
+     * it hasn't come in via a previously handled request that's being
+     * restarted for some reason.)
      */
     if (NULL != http->entry) {
        storeLockObject(http->entry);

--
You received this message because you are subscribed to the Google Groups 
"lusca-commit" group.
To post to this group, send email to [email protected].
To unsubscribe from this group, send email to 
[email protected].
For more options, visit this group at 
http://groups.google.com/group/lusca-commit?hl=en.

Reply via email to