- logback-dev - mailman3.qos.ch

[continuum] BUILD FAILURE: Logback-Parent
by John Noreplai 23 Jan '08

23 Jan '08

Online report : http://localhost:8090/continuum/servlet/continuum/target/ProjectBuild.vm/vi… Build statistics: State: Failed Previous State: Failed Started at: Wed, 23 Jan 2008 11:00:08 +0100 Finished at: Wed, 23 Jan 2008 11:00:15 +0100 Total time: 7s Build Trigger: Schedule Exit code: 1 Building machine hostname: pixie Operating system : Linux(unknown) Java version : 1.5.0_08(Sun Microsystems Inc.) Changes ceki - request parameters can be multi-valued as observed in bug 110 http://bugzilla.qos.ch/show_bug.cgi?id=110 /logback/trunk/logback-access/src/main/java/ch/qos/logback/access/pattern/RequestParameterConverter.java /logback/trunk/logback-access/src/main/java/ch/qos/logback/access/spi/AccessEvent.java **************************************************************************** Output: **************************************************************************** [INFO] Scanning for projects... [INFO] ---------------------------------------------------------------------------- [INFO] Building Logback-Parent [INFO] task-segment: [clean, install] [INFO] ---------------------------------------------------------------------------- [INFO] ------------------------------------------------------------------------ [ERROR] BUILD ERROR [INFO] ------------------------------------------------------------------------ [INFO] 'translate-project' was specified in an execution, but not found in the plugin [INFO] ------------------------------------------------------------------------ [INFO] For more information, run Maven with the -e switch [INFO] ------------------------------------------------------------------------ [INFO] Total time: 4 seconds [INFO] Finished at: Wed Jan 23 11:00:15 CET 2008 [INFO] Final Memory: 8M/83M [INFO] ------------------------------------------------------------------------ ****************************************************************************

1 0

svn commit: r1605 - in logback/trunk/logback-access/src/main/java/ch/qos/logback/access: pattern spi
by noreply.ceki＠qos.ch 23 Jan '08

23 Jan '08

Author: ceki Date: Wed Jan 23 10:52:19 2008 New Revision: 1605 Modified: logback/trunk/logback-access/src/main/java/ch/qos/logback/access/pattern/RequestParameterConverter.java logback/trunk/logback-access/src/main/java/ch/qos/logback/access/spi/AccessEvent.java Log: - request parameters can be multi-valued as observed in bug 110 http://bugzilla.qos.ch/show_bug.cgi?id=110 Modified: logback/trunk/logback-access/src/main/java/ch/qos/logback/access/pattern/RequestParameterConverter.java ============================================================================== --- logback/trunk/logback-access/src/main/java/ch/qos/logback/access/pattern/RequestParameterConverter.java (original) +++ logback/trunk/logback-access/src/main/java/ch/qos/logback/access/pattern/RequestParameterConverter.java Wed Jan 23 10:52:19 2008 @@ -1,9 +1,10 @@ package ch.qos.logback.access.pattern; +import java.util.Arrays; + import ch.qos.logback.access.spi.AccessEvent; import ch.qos.logback.core.util.OptionHelper; - public class RequestParameterConverter extends AccessConverter { String key; @@ -22,7 +23,14 @@ return "INACTIVE_REQUEST_PARAM_CONV"; } - return accessEvent.getRequestParameter(key); + String[] paramArray = accessEvent.getRequestParameter(key); + if (paramArray.length == 1) { + return paramArray[0]; + } else { + // for an array string {"a", "b"} named 'sa', Array.toString(sa) returns the string + // "[a, b]". + return Arrays.toString(paramArray); + } } } Modified: logback/trunk/logback-access/src/main/java/ch/qos/logback/access/spi/AccessEvent.java ============================================================================== --- logback/trunk/logback-access/src/main/java/ch/qos/logback/access/spi/AccessEvent.java (original) +++ logback/trunk/logback-access/src/main/java/ch/qos/logback/access/spi/AccessEvent.java Wed Jan 23 10:52:19 2008 @@ -14,6 +14,8 @@ import ch.qos.logback.access.pattern.AccessConverter; import ch.qos.logback.access.servlet.Util; +// Contributors: Joern Huxhorn (see also bug #110) + /** * The Access module's internal representation of logging events. When the * logging component instance is called in the container to log then a @@ -28,6 +30,8 @@ private static final long serialVersionUID = -3118194368414470960L; public final static String NA = "-"; + public final static String[] NA_STRING_ARRAY = new String[] {AccessEvent.NA}; + public final static String EMPTY = ""; public static final int SENTINEL = -1; @@ -46,7 +50,7 @@ String responseContent; Map<String, String> requestHeaderMap; - Map<String, Object> requestParameterMap; + Map<String, String[]> requestParameterMap; long contentLength = SENTINEL; int statusCode = SENTINEL; @@ -235,17 +239,24 @@ } public void buildRequestParameterMap() { - requestParameterMap = new HashMap<String, Object>(); + requestParameterMap = new HashMap<String, String[]>(); Enumeration e = httpRequest.getParameterNames(); if (e == null) { return; } while (e.hasMoreElements()) { String key = (String) e.nextElement(); - requestParameterMap.put(key, httpRequest.getParameter(key)); + requestParameterMap.put(key, httpRequest.getParameterValues(key)); } } + public Map<String, String[]> getRequestParameterMap() { + if (requestParameterMap == null) { + buildRequestParameterMap(); + } + return requestParameterMap; +} + public String getResponseHeader(String key) { return serverAdapter.getResponseHeader(key); } @@ -273,16 +284,16 @@ } } - public String getRequestParameter(String key) { + public String[] getRequestParameter(String key) { if (httpRequest != null) { - Object value = httpRequest.getParameter(key); + String[] value = httpRequest.getParameterValues(key); if (value == null) { - return AccessEvent.NA; + return NA_STRING_ARRAY; } else { - return value.toString(); + return value; } } else { - return AccessEvent.NA; + return NA_STRING_ARRAY; } }

1 0

Contributing patches
by Matt Fowles 21 Jan '08

21 Jan '08

All~ What is the preferred way to provide patches for new functionality or fixes? I emailed two patches directly to the list, but I could move them to tickets in bugzilla if people prefer. Matt

3 2

[Bug 122] New: evaluator on MDC conversion word
by bugzilla-daemon＠pixie.qos.ch 18 Jan '08

18 Jan '08

http://bugzilla.qos.ch/show_bug.cgi?id=122 Summary: evaluator on MDC conversion word Product: logback-classic Version: unspecified Platform: All OS/Version: All Status: NEW Severity: enhancement Priority: P3 Component: Other AssignedTo: logback-dev(a)qos.ch ReportedBy: hontvari3(a)solware.com It would be useful if the MDC keyword also has evaluator parameters just like the caller conversion word. An example usage: In a web application I store HTTP headers, parameters, URL into an MDC variable. This context information should be displayed with most log message. However there are a few types of log messages which are very frequent. Eventually (but it may take years) the reason for this messages are eliminated, but until then the long context information should be supressed from the log file. (In such cases the frequency or the existence of the event is the information, not the content of the event.) -- Configure bugmail: http://bugzilla.qos.ch/userprefs.cgi?tab=email ------- You are receiving this mail because: ------- You are the assignee for the bug, or are watching the assignee.

1 0

[Bug 57] New: Problems getting autogenerated Keys in Oracle with Oracle JDBC 10.2.03
by bugzilla-daemon＠pixie.qos.ch 11 Jan '08

11 Jan '08

http://bugzilla.qos.ch/show_bug.cgi?id=57 Summary: Problems getting autogenerated Keys in Oracle with Oracle JDBC 10.2.03 Product: logback-core Version: unspecified Platform: PC OS/Version: Windows Status: NEW Severity: critical Priority: P1 Component: Appender AssignedTo: logback-dev(a)qos.ch ReportedBy: klaus(a)ungers-online.com I've encountered troubles logging StackTrace/MDC entries with die DBAppender on our plattform! We are using Oracle 9.2.0.6.0 on a Solaris Box with Oracle JDBC Driver 10.2.0.3! I've debugged the processing of LogMessages into DBAppenderBase.append()! There I found a call "con.prepareStatement(getInsertSQL())" which is followed by a call "subAppend(...)" As far as I understand Oracle Drivers return generatedKeys only, if you use the right prepareStatement() methods, which is in my understanding the following: PreparedStatement prepareStatement(String sql, int autoGeneratedKeys) throws SQLException @see http://java.sun.com/j2se/1.5.0/docs/api/java/sql/Connection.html#prepareSta… which is available since JDK 1.4 -- Configure bugmail: http://bugzilla.qos.ch/userprefs.cgi?tab=email ------- You are receiving this mail because: ------- You are the assignee for the bug, or are watching the assignee.

1 4

[Bug 119] New: Make BasicStatusManager more sub-class friendly
by bugzilla-daemon＠pixie.qos.ch 10 Jan '08

10 Jan '08

http://bugzilla.qos.ch/show_bug.cgi?id=119 Summary: Make BasicStatusManager more sub-class friendly Product: logback-core Version: unspecified Platform: All OS/Version: All Status: NEW Severity: enhancement Priority: P2 Component: Other AssignedTo: logback-dev(a)qos.ch ReportedBy: anders(a)hammar.net To allow for more implementation specific sub-classes of BasicStatusManager, class attribute should be 'protected' (or add setters and getters). In the current implementation, a sub-class cannot (for instance) modify the statusList (to clear the statuses or such). -- Configure bugmail: http://bugzilla.qos.ch/userprefs.cgi?tab=email ------- You are receiving this mail because: ------- You are the assignee for the bug, or are watching the assignee.

1 0

[Bug 117] New: Synchronization of SimpleDateFormat object in DateConverter
by bugzilla-daemon＠pixie.qos.ch 10 Jan '08

10 Jan '08

http://bugzilla.qos.ch/show_bug.cgi?id=117 Summary: Synchronization of SimpleDateFormat object in DateConverter Product: logback-classic Version: unspecified Platform: All OS/Version: All Status: NEW Severity: normal Priority: P2 Component: Other AssignedTo: logback-dev(a)qos.ch ReportedBy: anders(a)hammar.net According to the API documentation of SimpleDateFormat, date formats are not synchronized and it is recommended to create separate format instances for each thread. However, a possibly simpler (mer efficient) solution would be to synchronize on the SimpleDateFormat object in the convert() method to prevent two threads to access it concurrently. -- Configure bugmail: http://bugzilla.qos.ch/userprefs.cgi?tab=email ------- You are receiving this mail because: ------- You are the assignee for the bug, or are watching the assignee.

1 4

[Bug 115] New: Improve WriterAppender
by bugzilla-daemon＠pixie.qos.ch 09 Jan '08

09 Jan '08

http://bugzilla.qos.ch/show_bug.cgi?id=115 Summary: Improve WriterAppender Product: logback-core Version: unspecified Platform: PC OS/Version: Windows Status: NEW Severity: enhancement Priority: P4 Component: Appender AssignedTo: logback-dev(a)qos.ch ReportedBy: anders(a)hammar.net Reviewing the ch.qos.logback.core.WriterAppender class, I see that the attributes are 'protected' (not 'private'). However, there are set and get methods for most of them (but not all). I suggest making the attributes private and create setters and getters for all of them (there is no getWriter() for instance). This would prevent subclasses from incorrectly working directly against the attributes (for instance, not using the setWriter() would cause errors). If some 'public' getters are not desirable, at least 'protected' ones should be provided. -- Configure bugmail: http://bugzilla.qos.ch/userprefs.cgi?tab=email ------- You are receiving this mail because: ------- You are the assignee for the bug, or are watching the assignee.

1 1

[Bug 116] New: Possibility to change StatusManger impl in subclass of ContextBase
by bugzilla-daemon＠pixie.qos.ch 09 Jan '08

09 Jan '08

http://bugzilla.qos.ch/show_bug.cgi?id=116 Summary: Possibility to change StatusManger impl in subclass of ContextBase Product: logback-core Version: unspecified Platform: PC OS/Version: Windows Status: NEW Severity: normal Priority: P3 Component: Other AssignedTo: logback-dev(a)qos.ch ReportedBy: anders(a)hammar.net I'd like the possibility to change the StatusManager implementation in a subclass of ch.qos.logback.core.ContextBase. As the sm attribute isn't 'protected' and there is no set method, this isn't possible without overriding the getStatusManger() method. A simple solution would be to add a 'protected' constructor that takes the StatusManager as argument. -- Configure bugmail: http://bugzilla.qos.ch/userprefs.cgi?tab=email ------- You are receiving this mail because: ------- You are the assignee for the bug, or are watching the assignee.

1 2

Unit Test Fix
by Matt Fowles 08 Jan '08

08 Jan '08

All~ The attached patch fixes what a small typo in a unit test. The test still fails, but now it fails for good reasons. Matt

1 0