The current viewlog CGI needs to be modified to be able to work properly with the Console. The CGI currently only supports the GET request method. This works for Admin Express, but Console expects it to support the POST request method. The output of the CGI currently includes a log selection combo box as well as other HTML elements that make for a nice useful log viewer page in Admin Express. The Console needs to be able to get get the raw log content that it requests. The attached diffs address the above issues. With these changes, the CGI will work with both the GET and POST request methods. I've added a new CGI parameter called "nohtml". This optional parameter is a flag that allows the caller to ask for only the raw content to be sent back. This satisfies the needs of the Console while keeping the behavior the same for Admin Express.
Created attachment 158068 [details] CVS Diffs
Your diffs look good.
Checked into HEAD. Thanks for the review Noriko! Checking in admserv/cgi-src40/viewlog.c; /cvs/dirsec/adminserver/admserv/cgi-src40/viewlog.c,v <-- viewlog.c new revision: 1.9; previous revision: 1.8 done Checking in admserv/cgi-src40/viewlog.properties; /cvs/dirsec/adminserver/admserv/cgi-src40/viewlog.properties,v <-- viewlog.properties new revision: 1.5; previous revision: 1.4 done Checking in admserv/schema/ldif/11dstasks.ldif.tmpl; /cvs/dirsec/adminserver/admserv/schema/ldif/11dstasks.ldif.tmpl,v <-- 11dstasks.ldif.tmpl new revision: 1.3; previous revision: 1.2 done