Bug 59924 - The log level of XXX package is set to DEBUG if log_level.XXXX property value contains spaces, same for __log function
Summary: The log level of XXX package is set to DEBUG if log_level.XXXX property value...
Status: RESOLVED FIXED
Alias: None
Product: JMeter - Now in Github
Classification: Unclassified
Component: Main (show other bugs)
Version: 3.0
Hardware: All All
: P2 enhancement (vote)
Target Milestone: ---
Assignee: JMeter issues mailing list
URL:
Keywords:
Depends on:
Blocks:
 
Reported: 2016-08-02 08:16 UTC by sc.yiwei
Modified: 2016-08-02 20:27 UTC (History)
1 user (show)


Attachments
Add an attachment (proposed patch, testcase, etc.)

Note You need to log in before you can comment on or make changes to this bug.
Description sc.yiwei 2016-08-02 08:16:14 UTC 
For example, when I set the jmeter log_levl with extra spaces before or after INFO,the log_level for org.apache.jmeter will be set to debug.
log_level.jmeter= INFO 

Do you consider add the trim() function for the appProperties? Thanks.
@@ -208,7 +208,7 @@ public final class LoggingManager {
             // don't match the empty category
             {
                 String category = prop.substring(LOG_PRIORITY.length() + 1);
-                setPriority(appProperties.getProperty(prop), category);
+                setPriority(appProperties.getProperty(prop).trim(), category);
             }
             if (prop.startsWith(LOG_FILE + ".")) { //$NON_NLS-1$
                 String category = prop.substring(LOG_FILE.length() + 1);
Comment 1 Philippe Mouawad 2016-08-02 20:27:41 UTC 
Author: pmouawad
Date: Tue Aug  2 20:27:19 2016
New Revision: 1755002

URL: http://svn.apache.org/viewvc?rev=1755002&view=rev
Log:
Bug 59924 - The log level of XXX package is set to DEBUG if log_level.XXXX property value contains spaces, same for __log function
Bugzilla Id: 59924

Modified:
    jmeter/trunk/src/functions/org/apache/jmeter/functions/LogFunction.java
    jmeter/trunk/src/jorphan/org/apache/jorphan/logging/LoggingManager.java
    jmeter/trunk/xdocs/changes.xml
Comment 2 The ASF infrastructure team 2022-09-24 20:38:04 UTC 
This issue has been migrated to GitHub: https://github.com/apache/jmeter/issues/4034