You are viewing a plain text version of this content. The canonical link for it is here.
Posted to issues@struts.apache.org by "zhouyanming (JIRA)" <ji...@apache.org> on 2012/06/11 05:17:43 UTC

[jira] [Created] (WW-3840) correct JakartaMultiPartRequest LOG

zhouyanming created WW-3840:
-------------------------------

             Summary: correct JakartaMultiPartRequest LOG
                 Key: WW-3840
                 URL: https://issues.apache.org/jira/browse/WW-3840
             Project: Struts 2
          Issue Type: Bug
            Reporter: zhouyanming
            Priority: Minor


static final Logger LOG = LoggerFactory.getLogger(MultiPartRequest.class);
should be
static final Logger LOG = LoggerFactory.getLogger(JakartaMultiPartRequest.class);

like
protected static final Logger LOG = LoggerFactory.getLogger(MultiPartRequestWrapper.class);
in MultiPartRequestWrapper.java

--
This message is automatically generated by JIRA.
If you think it was sent incorrectly, please contact your JIRA administrators: https://issues.apache.org/jira/secure/ContactAdministrators!default.jspa
For more information on JIRA, see: http://www.atlassian.com/software/jira

        

[jira] [Updated] (WW-3840) correct JakartaMultiPartRequest LOG and change cleanUp log level to DEBUG

Posted by "zhouyanming (JIRA)" <ji...@apache.org>.
     [ https://issues.apache.org/jira/browse/WW-3840?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ]

zhouyanming updated WW-3840:
----------------------------

    Description: 
static final Logger LOG = LoggerFactory.getLogger(MultiPartRequest.class);
should be
static final Logger LOG = LoggerFactory.getLogger(JakartaMultiPartRequest.class);

like
protected static final Logger LOG = LoggerFactory.getLogger(MultiPartRequestWrapper.class);
in MultiPartRequestWrapper.java


in JakartaMultiPartRequest.cleanUp(),info level log is annoying,please change to debug

 if (LOG.isDebugEnabled()) {
                    String msg = LocalizedTextUtil.findText(this.getClass(), "struts.messages.removing.file",
                            Locale.ENGLISH, "no.message.found", new Object[]{name, item});
                    LOG.debug(msg);
                }

  was:
static final Logger LOG = LoggerFactory.getLogger(MultiPartRequest.class);
should be
static final Logger LOG = LoggerFactory.getLogger(JakartaMultiPartRequest.class);

like
protected static final Logger LOG = LoggerFactory.getLogger(MultiPartRequestWrapper.class);
in MultiPartRequestWrapper.java

        Summary: correct JakartaMultiPartRequest LOG and change cleanUp log level to DEBUG   (was: correct JakartaMultiPartRequest LOG)
    
> correct JakartaMultiPartRequest LOG and change cleanUp log level to DEBUG 
> --------------------------------------------------------------------------
>
>                 Key: WW-3840
>                 URL: https://issues.apache.org/jira/browse/WW-3840
>             Project: Struts 2
>          Issue Type: Bug
>            Reporter: zhouyanming
>            Priority: Minor
>
> static final Logger LOG = LoggerFactory.getLogger(MultiPartRequest.class);
> should be
> static final Logger LOG = LoggerFactory.getLogger(JakartaMultiPartRequest.class);
> like
> protected static final Logger LOG = LoggerFactory.getLogger(MultiPartRequestWrapper.class);
> in MultiPartRequestWrapper.java
> in JakartaMultiPartRequest.cleanUp(),info level log is annoying,please change to debug
>  if (LOG.isDebugEnabled()) {
>                     String msg = LocalizedTextUtil.findText(this.getClass(), "struts.messages.removing.file",
>                             Locale.ENGLISH, "no.message.found", new Object[]{name, item});
>                     LOG.debug(msg);
>                 }

--
This message is automatically generated by JIRA.
If you think it was sent incorrectly, please contact your JIRA administrators: https://issues.apache.org/jira/secure/ContactAdministrators!default.jspa
For more information on JIRA, see: http://www.atlassian.com/software/jira

        

[jira] [Resolved] (WW-3840) correct JakartaMultiPartRequest LOG and change cleanUp log level to DEBUG

Posted by "Lukasz Lenart (JIRA)" <ji...@apache.org>.
     [ https://issues.apache.org/jira/browse/WW-3840?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ]

Lukasz Lenart resolved WW-3840.
-------------------------------

    Resolution: Fixed
      Assignee: Lukasz Lenart

Solved, thanks for reporting!
                
> correct JakartaMultiPartRequest LOG and change cleanUp log level to DEBUG 
> --------------------------------------------------------------------------
>
>                 Key: WW-3840
>                 URL: https://issues.apache.org/jira/browse/WW-3840
>             Project: Struts 2
>          Issue Type: Bug
>            Reporter: zhouyanming
>            Assignee: Lukasz Lenart
>            Priority: Minor
>             Fix For: 2.3.5
>
>
> static final Logger LOG = LoggerFactory.getLogger(MultiPartRequest.class);
> should be
> static final Logger LOG = LoggerFactory.getLogger(JakartaMultiPartRequest.class);
> like
> protected static final Logger LOG = LoggerFactory.getLogger(MultiPartRequestWrapper.class);
> in MultiPartRequestWrapper.java
> in JakartaMultiPartRequest.cleanUp(),info level log is annoying,please change to debug
>  if (LOG.isDebugEnabled()) {
>                     String msg = LocalizedTextUtil.findText(this.getClass(), "struts.messages.removing.file",
>                             Locale.ENGLISH, "no.message.found", new Object[]{name, item});
>                     LOG.debug(msg);
>                 }

--
This message is automatically generated by JIRA.
If you think it was sent incorrectly, please contact your JIRA administrators
For more information on JIRA, see: http://www.atlassian.com/software/jira

[jira] [Updated] (WW-3840) correct JakartaMultiPartRequest LOG and change cleanUp log level to DEBUG

Posted by "Lukasz Lenart (JIRA)" <ji...@apache.org>.
     [ https://issues.apache.org/jira/browse/WW-3840?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ]

Lukasz Lenart updated WW-3840:
------------------------------

    Fix Version/s: 2.3.5
    
> correct JakartaMultiPartRequest LOG and change cleanUp log level to DEBUG 
> --------------------------------------------------------------------------
>
>                 Key: WW-3840
>                 URL: https://issues.apache.org/jira/browse/WW-3840
>             Project: Struts 2
>          Issue Type: Bug
>            Reporter: zhouyanming
>            Priority: Minor
>             Fix For: 2.3.5
>
>
> static final Logger LOG = LoggerFactory.getLogger(MultiPartRequest.class);
> should be
> static final Logger LOG = LoggerFactory.getLogger(JakartaMultiPartRequest.class);
> like
> protected static final Logger LOG = LoggerFactory.getLogger(MultiPartRequestWrapper.class);
> in MultiPartRequestWrapper.java
> in JakartaMultiPartRequest.cleanUp(),info level log is annoying,please change to debug
>  if (LOG.isDebugEnabled()) {
>                     String msg = LocalizedTextUtil.findText(this.getClass(), "struts.messages.removing.file",
>                             Locale.ENGLISH, "no.message.found", new Object[]{name, item});
>                     LOG.debug(msg);
>                 }

--
This message is automatically generated by JIRA.
If you think it was sent incorrectly, please contact your JIRA administrators: https://issues.apache.org/jira/secure/ContactAdministrators!default.jspa
For more information on JIRA, see: http://www.atlassian.com/software/jira

        

[jira] [Commented] (WW-3840) correct JakartaMultiPartRequest LOG and change cleanUp log level to DEBUG

Posted by "Hudson (JIRA)" <ji...@apache.org>.
    [ https://issues.apache.org/jira/browse/WW-3840?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=13454351#comment-13454351 ] 

Hudson commented on WW-3840:
----------------------------

Integrated in Struts2 #531 (See [https://builds.apache.org/job/Struts2/531/])
    WW-3840 corrects LOG and reduce log level to debug inside cleanUp() method (Revision 1384107)

     Result = SUCCESS
lukaszlenart : 
Files : 
* /struts/struts2/trunk/core/src/main/java/org/apache/struts2/dispatcher/multipart/JakartaMultiPartRequest.java

                
> correct JakartaMultiPartRequest LOG and change cleanUp log level to DEBUG 
> --------------------------------------------------------------------------
>
>                 Key: WW-3840
>                 URL: https://issues.apache.org/jira/browse/WW-3840
>             Project: Struts 2
>          Issue Type: Bug
>            Reporter: zhouyanming
>            Assignee: Lukasz Lenart
>            Priority: Minor
>             Fix For: 2.3.5
>
>
> static final Logger LOG = LoggerFactory.getLogger(MultiPartRequest.class);
> should be
> static final Logger LOG = LoggerFactory.getLogger(JakartaMultiPartRequest.class);
> like
> protected static final Logger LOG = LoggerFactory.getLogger(MultiPartRequestWrapper.class);
> in MultiPartRequestWrapper.java
> in JakartaMultiPartRequest.cleanUp(),info level log is annoying,please change to debug
>  if (LOG.isDebugEnabled()) {
>                     String msg = LocalizedTextUtil.findText(this.getClass(), "struts.messages.removing.file",
>                             Locale.ENGLISH, "no.message.found", new Object[]{name, item});
>                     LOG.debug(msg);
>                 }

--
This message is automatically generated by JIRA.
If you think it was sent incorrectly, please contact your JIRA administrators
For more information on JIRA, see: http://www.atlassian.com/software/jira