Uploaded image for project: 'Struts 2'
  1. Struts 2
  2. WW-4839

JakartaStreamMultiPartRequest Should Honor "struts.multipart.maxSize"

    Details

      Description

      The JakartaStreamMultiPartRequest does not honor the maximum request size configuration in the struts.multipart.maxSize property when ServletRequest.getContentLength() returns -1.

      Note: ServletRequest.getContentLengthLong() is available in Servlet API 3.1.

        Issue Links

          Activity

          Hide
          githubbot ASF GitHub Bot added a comment -

          GitHub user swiftelan opened a pull request:

          https://github.com/apache/struts/pull/160

          WW-4839 JakartaStreamMultiPartRequest Should Honor "struts.multipart.…

          WW-4839(https://issues.apache.org/jira/browse/WW-4839) JakartaStreamMultiPartRequest Should Honor "struts.multipart.maxSize" when content length is unknown.

          You can merge this pull request into a Git repository by running:

          $ git pull https://github.com/swiftelan/struts WW-4839

          Alternatively you can review and apply these changes as the patch at:

          https://github.com/apache/struts/pull/160.patch

          To close this pull request, make a commit to your master/trunk branch
          with (at least) the following in the commit message:

          This closes #160


          commit eb0137168f0c0e09a1ef6532965ffc1634a36291
          Author: Aaron Johnson <johnson.aaron5@gmail.com>
          Date: 2017-08-04T20:58:23Z

          WW-4839 JakartaStreamMultiPartRequest Should Honor "struts.multipart.maxSize"


          Show
          githubbot ASF GitHub Bot added a comment - GitHub user swiftelan opened a pull request: https://github.com/apache/struts/pull/160 WW-4839 JakartaStreamMultiPartRequest Should Honor "struts.multipart.… WW-4839 ( https://issues.apache.org/jira/browse/WW-4839 ) JakartaStreamMultiPartRequest Should Honor "struts.multipart.maxSize" when content length is unknown. You can merge this pull request into a Git repository by running: $ git pull https://github.com/swiftelan/struts WW-4839 Alternatively you can review and apply these changes as the patch at: https://github.com/apache/struts/pull/160.patch To close this pull request, make a commit to your master/trunk branch with (at least) the following in the commit message: This closes #160 commit eb0137168f0c0e09a1ef6532965ffc1634a36291 Author: Aaron Johnson <johnson.aaron5@gmail.com> Date: 2017-08-04T20:58:23Z WW-4839 JakartaStreamMultiPartRequest Should Honor "struts.multipart.maxSize"
          Hide
          jira-bot ASF subversion and git services added a comment -

          Commit eb0137168f0c0e09a1ef6532965ffc1634a36291 in struts's branch refs/heads/master from Aaron Johnson
          [ https://git-wip-us.apache.org/repos/asf?p=struts.git;h=eb01371 ]

          WW-4839 JakartaStreamMultiPartRequest Should Honor "struts.multipart.maxSize"

          Show
          jira-bot ASF subversion and git services added a comment - Commit eb0137168f0c0e09a1ef6532965ffc1634a36291 in struts's branch refs/heads/master from Aaron Johnson [ https://git-wip-us.apache.org/repos/asf?p=struts.git;h=eb01371 ] WW-4839 JakartaStreamMultiPartRequest Should Honor "struts.multipart.maxSize"
          Hide
          jira-bot ASF subversion and git services added a comment -

          Commit ff6148795ac52e6de5b80c01e7a1a0cc4da61857 in struts's branch refs/heads/master from Lukasz Lenart
          [ https://git-wip-us.apache.org/repos/asf?p=struts.git;h=ff61487 ]

          WW-4839 JakartaStreamMultiPartRequest honors struts.multipart.maxSize

          Show
          jira-bot ASF subversion and git services added a comment - Commit ff6148795ac52e6de5b80c01e7a1a0cc4da61857 in struts's branch refs/heads/master from Lukasz Lenart [ https://git-wip-us.apache.org/repos/asf?p=struts.git;h=ff61487 ] WW-4839 JakartaStreamMultiPartRequest honors struts.multipart.maxSize
          Hide
          githubbot ASF GitHub Bot added a comment -

          Github user asfgit closed the pull request at:

          https://github.com/apache/struts/pull/160

          Show
          githubbot ASF GitHub Bot added a comment - Github user asfgit closed the pull request at: https://github.com/apache/struts/pull/160
          Hide
          lukaszlenart Lukasz Lenart added a comment -

          PR got merged, thanks a lot!

          Show
          lukaszlenart Lukasz Lenart added a comment - PR got merged, thanks a lot!

            People

            • Assignee:
              Unassigned
              Reporter:
              swiftelan Aaron Johnson
            • Votes:
              0 Vote for this issue
              Watchers:
              4 Start watching this issue

              Dates

              • Created:
                Updated:
                Resolved:

                Development