[BUG] appsession lookup in URL does not work

We've been trying to use the latest release (1.3.14.2) of haproxy  to do
sticky sessions.  Cookie insertion is not an option for us, although we
would much rather use it, as we are trying to work around a problem where
cookies are unreliable.  The appsession functionality only partially worked
(it wouldn't read the session id out of a query string) until we made the
following code change to the get_srv_from_appsession function in
proto_http.c.
This commit is contained in:
Ryan Warnick 2008-02-17 11:24:35 +01:00 committed by Willy Tarreau
parent 8b2f55aec4
commit 6d0b1fac23

View File

@ -4561,7 +4561,8 @@ void manage_server_side_cookies(struct session *t, struct buffer *rtr)
asession_temp->serverid = NULL;
/* only do insert, if lookup fails */
if (appsession_hash_lookup(&(t->be->htbl_proxy), asession_temp->sessid) == NULL) {
asession_temp = appsession_hash_lookup(&(t->be->htbl_proxy), asession_temp->sessid);
if (asession_temp == NULL) {
if ((asession_temp = pool_alloc2(pool2_appsess)) == NULL) {
Alert("Not enough Memory process_srv():asession:calloc().\n");
send_log(t->be, LOG_ALERT, "Not enough Memory process_srv():asession:calloc().\n");
@ -4732,7 +4733,8 @@ void get_srv_from_appsession(struct session *t, const char *begin, int len)
asession_temp->serverid = NULL;
/* only do insert, if lookup fails */
if (appsession_hash_lookup(&(t->be->htbl_proxy), asession_temp->sessid) == NULL) {
asession_temp = appsession_hash_lookup(&(t->be->htbl_proxy), asession_temp->sessid);
if (asession_temp == NULL) {
if ((asession_temp = pool_alloc2(pool2_appsess)) == NULL) {
/* free previously allocated memory */
pool_free2(apools.sessid, local_asession.sessid);