Bug 32963 - lb setAttribute() spews bogus errors
Summary: lb setAttribute() spews bogus errors
Status: RESOLVED WONTFIX
Alias: None
Product: Tomcat 4
Classification: Unclassified
Component: Connector:Coyote JK 2 (deprecated) (show other bugs)
Version: 4.1.31
Hardware: Macintosh All
: P2 trivial (vote)
Target Milestone: ---
Assignee: Tomcat Developers Mailing List
URL:
Keywords:
Depends on:
Blocks:
 
Reported: 2005-01-06 14:09 UTC by Moriyoshi Koizumi
Modified: 2005-01-06 05:16 UTC (History)
0 users



Attachments

Note You need to log in before you can comment on or make changes to this bug.
Description Moriyoshi Koizumi 2005-01-06 14:09:27 UTC
Because necessary "return JK_OK"s are missing all over the place in setAttribute(), setting any attribute 
other than worker ends up with a bogus error message like the following:

[Thu Jan 06 22:06:31 2005] [notice] config.setAttribute() Error setting lb: stickySession 1
[Thu Jan 06 22:06:31 2005] [notice] config.update(): done lb:


    else if (strcmp(name, "attempts") == 0) {
        lb_priv->attempts = atoi(value);
<< return JK_OK SHOULD HAVE BEEN HERE >>
    }
    return JK_ERR;
}
Comment 1 Mladen Turk 2005-01-06 14:16:17 UTC
JK2 is deprecated.
http://jakarta.apache.org/tomcat/connectors-doc/news/20041100.html#20041115.1