mturk 2005/02/22 01:40:12 Modified: jk/native/common jk_status.c Log: Be more dtd conformant. Patch provided by Peter Rossbach. Revision Changes Path 1.24 +58 -48 jakarta-tomcat-connectors/jk/native/common/jk_status.c Index: jk_status.c =================================================================== RCS file: /home/cvs/jakarta-tomcat-connectors/jk/native/common/jk_status.c,v retrieving revision 1.23 retrieving revision 1.24 diff -u -r1.23 -r1.24 --- jk_status.c 21 Feb 2005 16:40:45 -0000 1.23 +++ jk_status.c 22 Feb 2005 09:40:12 -0000 1.24 @@ -323,7 +323,7 @@ { unsigned int i; - jk_puts(s, "<br />Uri Mappings:\n"); + jk_puts(s, "<br/>Uri Mappings:\n"); jk_puts(s, "<table>\n<tr><th>Match Type</th><th>Uri</th>" "<th>Context</th><th>Suffix</th></tr>\n"); for (i = 0; i < uwmap->size; i++) { @@ -336,7 +336,11 @@ "</td><td>", NULL); jk_puts(s, uwr->uri); jk_putv(s, "</td><td>", uwr->context, NULL); - jk_putv(s, "</td><td>", uwr->suffix, NULL); + if (uwr->suffix) + jk_putv(s, "</td><td>", uwr->suffix, NULL); + else + jk_putv(s, "</td><td>", " ", NULL); + jk_puts(s, "</td></tr>\n"); } jk_puts(s, "</table>\n"); @@ -380,7 +384,7 @@ /* Skip status, jni and ajp12 worker */ continue; } - jk_puts(s, "<hr />\n<h3>Worker Status for "); + jk_puts(s, "<hr/>\n<h3>Worker Status for "); if (dworker && strcmp(dworker, sw->we->worker_list[i]) == 0) { /* Next click will colapse the editor */ jk_putv(s, "<a href=\"", s->req_uri, "?cmd=show\">", NULL); @@ -392,7 +396,7 @@ if (lb != NULL) { unsigned int j; int selected = -1; - jk_puts(s, "<table border=\"0\"><tr>" + jk_puts(s, "<table><tr>" "<th>Type</th><th>Sticky session</th>" "<th>Force Sticky session</th>" "<th>Retries</th>" @@ -403,8 +407,8 @@ jk_putv(s, "<td>", status_val_bool(lb->s->sticky_session_force), "</td>", NULL); jk_printf(s, "<td>%d</td>", lb->s->retries); - jk_puts(s, "</tr>\n</table>\n"); - jk_puts(s, "<table border=\"0\"><tr>" + jk_puts(s, "</tr>\n</table>\n<br/>\n"); + jk_puts(s, "<table><tr>" "<th>Name</th><th>Type</th><th>Host</th><th>Addr</th>" "<th>Stat</th><th>F</th><th>V</th><th>Acc</th><th>Err</th>" "<th>Wr</th><th>Rd</th><th>Busy</th><th>RR</th><th>Cd</th></tr>\n"); @@ -414,7 +418,7 @@ jk_putv(s, "<tr>\n<td><a href=\"", s->req_uri, "?cmd=show&w=", wr->s->name, "\">", - wr->s->name, "</td>", NULL); + wr->s->name, "</a></td>", NULL); if (dworker && strcmp(dworker, wr->s->name) == 0) selected = j; jk_putv(s, "<td>", status_worker_type(wr->w->type), "</td>", NULL); @@ -437,76 +441,82 @@ jk_putv(s, "<td>", status_strfsize(wr->s->readed, buf), "</td>", NULL); jk_printf(s, "<td>%u</td><td>", wr->s->busy); - jk_puts(s, wr->s->redirect); + if (wr->s->redirect && *wr->s->redirect) + jk_puts(s, wr->s->redirect); + else + jk_puts(s," "); jk_puts(s, "</td><td>\n"); - jk_puts(s, wr->s->domain); + if (wr->s->domain && *wr->s->domain) + jk_puts(s, wr->s->domain); + else + jk_puts(s," "); jk_puts(s, "</td>\n</tr>\n"); } - jk_puts(s, "</table><br />\n"); + jk_puts(s, "</table><br/>\n"); if (selected >= 0) { worker_record_t *wr = &(lb->lb_workers[selected]); - jk_putv(s, "<hr /><h3>Edit worker settings for ", + jk_putv(s, "<hr/><h3>Edit worker settings for ", wr->s->name, "</h3>\n", NULL); jk_putv(s, "<form method=\"GET\" action=\"", s->req_uri, "\">\n", NULL); - jk_puts(s, "<table>\n<input type=hidden name=\"cmd\" "); + jk_puts(s, "<table>\n<input type=\"hidden\" name=\"cmd\" "); jk_puts(s, "value=\"update\">\n"); - jk_puts(s, "<input type=hidden name=\"w\" "); + jk_puts(s, "<input type=\"hidden\" name=\"w\" "); jk_putv(s, "value=\"", wr->s->name, "\">\n", NULL); - jk_puts(s, "<input type=hidden name=\"id\" "); + jk_puts(s, "<input type=\"hidden\" name=\"id\" "); jk_printf(s, "value=\"%u\">\n</table>\n", selected); - jk_puts(s, "<input type=hidden name=\"lb\" "); + jk_puts(s, "<input type=\"hidden\" name=\"lb\" "); jk_printf(s, "value=\"%u\">\n</table>\n", i); - jk_puts(s, "<table>\n<tr><td>Load factor:</td><td><input name=\"wf\" type=text "); - jk_printf(s, "value=\"%d\"></td><tr>\n", wr->s->lb_factor); - jk_puts(s, "<tr><td>Route Redirect:</td><td><input name=\"wr\" type=text "); + jk_puts(s, "<table>\n<tr><td>Load factor:</td><td><input name=\"wf\" type=\"text\" "); + jk_printf(s, "value=\"%d\"/></td><tr>\n", wr->s->lb_factor); + jk_puts(s, "<tr><td>Route Redirect:</td><td><input name=\"wr\" type=\"text\" "); jk_putv(s, "value=\"", wr->s->redirect, NULL); - jk_puts(s, "\"></td></tr>\n"); - jk_puts(s, "<tr><td>Cluster Domain:</td><td><input name=\"wc\" type=text "); + jk_puts(s, "\"/></td></tr>\n"); + jk_puts(s, "<tr><td>Cluster Domain:</td><td><input name=\"wc\" type=\"text\" "); jk_putv(s, "value=\"", wr->s->domain, NULL); - jk_puts(s, "\"></td></tr>\n"); - jk_puts(s, "<tr><td>Disabled:</td><td><input name=\"wd\" type=checkbox"); + jk_puts(s, "\"/></td></tr>\n"); + jk_puts(s, "<tr><td>Disabled:</td><td><input name=\"wd\" type=\"checkbox\""); if (wr->s->is_disabled) - jk_puts(s, " checked"); - jk_puts(s, "></td></tr>\n"); + jk_puts(s, " value=\"checked\""); + jk_puts(s, "/></td></tr>\n"); jk_puts(s, "</td></tr>\n</table>\n"); - jk_puts(s, "<br /><input type=submit value=\"Update Worker\">\n</form>\n"); + jk_puts(s, "<br/><input type=\"submit\" value=\"Update Worker\"/>\n</form>\n"); } else if (dworker && strcmp(dworker, sw->we->worker_list[i]) == 0) { /* Edit Load balancer settings */ - jk_putv(s, "<hr /><h3>Edit Load balancer settings for ", + jk_putv(s, "<hr/><h3>Edit Load balancer settings for ", dworker, "</h3>\n", NULL); jk_putv(s, "<form method=\"GET\" action=\"", s->req_uri, "\">\n", NULL); - jk_puts(s, "<table>\n<input type=hidden name=\"cmd\" "); - jk_puts(s, "value=\"update\">\n"); - jk_puts(s, "<input type=hidden name=\"w\" "); - jk_putv(s, "value=\"", dworker, "\">\n", NULL); - jk_puts(s, "<input type=hidden name=\"id\" "); - jk_printf(s, "value=\"%u\">\n</table>\n", i); - - jk_puts(s, "<table>\n<tr><td>Retries :</td><td><input name=\"lr\" type=text "); - jk_printf(s, "value=\"%d\"></td><tr>\n", lb->s->retries); - jk_puts(s, "<tr><td>Recover time :</td><td><input name=\"lt\" type=text "); - jk_printf(s, "value=\"%d\"></td><tr>\n", lb->s->recover_wait_time); - jk_puts(s, "<tr><td>Sticky session:</td><td><input name=\"ls\" type=checkbox"); + jk_puts(s, "<table>\n<input type=\"hidden\" name=\"cmd\" "); + jk_puts(s, "value=\"update\"/>\n"); + jk_puts(s, "<input type=\"hidden\" name=\"w\" "); + jk_putv(s, "value=\"", dworker, "\"/>\n", NULL); + jk_puts(s, "<input type=\"hidden\" name=\"id\" "); + jk_printf(s, "value=\"%u\"/>\n</table>\n", i); + + jk_puts(s, "<table>\n<tr><td>Retries:</td><td><input name=\"lr\" type=\"text\" "); + jk_printf(s, "value=\"%d\"/></td></tr>\n", lb->s->retries); + jk_puts(s, "<tr><td>Recover time:</td><td><input name=\"lt\" type=\"text\" "); + jk_printf(s, "value=\"%d\"/></td></tr>\n", lb->s->recover_wait_time); + jk_puts(s, "<tr><td>Sticky session:</td><td><input name=\"ls\" type=\"checkbox\""); if (lb->s->sticky_session) - jk_puts(s, " checked"); - jk_puts(s, "></td></tr>\n"); - jk_puts(s, "<tr><td>Force Sticky session:</td><td><input name=\"lf\" type=checkbox"); + jk_puts(s, " value=\"checked\""); + jk_puts(s, "/></td></tr>\n"); + jk_puts(s, "<tr><td>Force Sticky session:</td><td><input name=\"lf\" type=\"checkbox\""); if (lb->s->sticky_session_force) - jk_puts(s, " checked"); - jk_puts(s, "></td></tr>\n"); - jk_puts(s, "</td></tr>\n</table>\n"); + jk_puts(s, " value=\"checked\""); + jk_puts(s, "/></td></tr>\n"); + jk_puts(s, "</table>\n"); display_maps(s, sw, s->uw_map, dworker, l); - jk_puts(s, "<br /><input type=submit value=\"Update Balancer\"></form>\n"); + jk_puts(s, "<br/><input type=\"submit\" value=\"Update Balancer\"/></form>\n"); } } else { - jk_puts(s, "\n\n<table border=\"0\"><tr>" + jk_puts(s, "\n\n<table><tr>" "<th>Type</th><th>Host</th><th>Addr</th>" "</tr>\n<tr>"); jk_putv(s, "<td>", status_worker_type(w->type), "</td>", NULL); @@ -519,7 +529,7 @@ } } /* Display legend */ - jk_puts(s, "<hr /><table>\n" + jk_puts(s, "<hr/><table>\n" "<tr><th>Name</th><td>Worker route name</td></tr>\n" "<tr><th>Type</th><td>Worker type</td></tr>\n" "<tr><th>Addr</th><td>Backend Address info</td></tr>\n"
--------------------------------------------------------------------- To unsubscribe, e-mail: [EMAIL PROTECTED] For additional commands, e-mail: [EMAIL PROTECTED]