Bug 246020 - Viewlog CGI needs to be modified to work with Console
Summary: Viewlog CGI needs to be modified to work with Console
Alias: None
Product: 389
Classification: Retired
Component: Admin   
(Show other bugs)
Version: 1.1.0
Hardware: All
OS: Linux
Target Milestone: ---
Assignee: Nathan Kinder
QA Contact: Viktor Ashirov
Depends On:
Blocks: 240316 FDS1.1.0
TreeView+ depends on / blocked
Reported: 2007-06-27 21:45 UTC by Nathan Kinder
Modified: 2015-12-07 17:04 UTC (History)
0 users

Fixed In Version:
Doc Type: Bug Fix
Doc Text:
Story Points: ---
Clone Of:
Last Closed: 2015-12-07 17:04:27 UTC
Type: ---
Regression: ---
Mount Type: ---
Documentation: ---
Verified Versions:
Category: ---
oVirt Team: ---
RHEL 7.3 requirements from Atomic Host:
Cloudforms Team: ---

Attachments (Terms of Use)
CVS Diffs (10.76 KB, patch)
2007-06-27 21:45 UTC, Nathan Kinder
no flags Details | Diff

Description Nathan Kinder 2007-06-27 21:45:59 UTC
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

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.

Comment 1 Nathan Kinder 2007-06-27 21:45:59 UTC
Created attachment 158068 [details]
CVS Diffs

Comment 2 Noriko Hosoi 2007-06-27 22:18:53 UTC
Your diffs look good.

Comment 3 Nathan Kinder 2007-06-27 22:25:44 UTC
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
Checking in admserv/cgi-src40/viewlog.properties;
/cvs/dirsec/adminserver/admserv/cgi-src40/viewlog.properties,v  <-- 
new revision: 1.5; previous revision: 1.4
Checking in admserv/schema/ldif/11dstasks.ldif.tmpl;
/cvs/dirsec/adminserver/admserv/schema/ldif/11dstasks.ldif.tmpl,v  <-- 
new revision: 1.3; previous revision: 1.2

Note You need to log in before you can comment on or make changes to this bug.