Le lundi 12 octobre 2009 23:17:43, Aleksandar Lazic a écrit :
> Hi Cyril,
>
> good catch.
>
> >+}
> >+
> >+/*
> >-#if defined(DEBUG_HASH)
> >- Alert("manage_client_side_cookies\n");
> >-
> >appsession_hash_dump(&(t->be->htbl_proxy));
> >-#endif
>
> After a quick look I think it would be nice to dump the sessions also
> in the client.
Yes, you're right, I missed it after several tests on different snapshots.
Here comes a second patch to reintroduce these debug lines :
diff -Naur haproxy-1.4-dev4-appsession/src/proto_http.c
haproxy-1.4-dev4-appsession2/src/proto_http.c
--- haproxy-1.4-dev4-appsession/src/proto_http.c 2009-10-12
21:56:38.000000000 +0200
+++ haproxy-1.4-dev4-appsession2/src/proto_http.c 2009-10-13
21:26:19.000000000 +0200
@@ -3862,6 +3862,10 @@
/* Cool... it's the right one */
manage_client_side_appsession(t, p3);
+#if defined(DEBUG_HASH)
+ Alert("manage_client_side_cookies\n");
+
appsession_hash_dump(&(t->be->htbl_proxy));
+#endif
}/* end if ((t->proxy->appsession_name != NULL)
... */
}
@@ -4450,6 +4454,10 @@
request_line += t->be->appsession_name_len + 1;
manage_client_side_appsession(t, request_line);
+#if defined(DEBUG_HASH)
+ Alert("get_srv_from_appsession\n");
+ appsession_hash_dump(&(t->be->htbl_proxy));
+#endif
}
--
Cyril Bonté
