ICEfaces
  1. ICEfaces
  2. ICE-4703

file upload hang from iPhone (ice:inputFile)

    Details

    • Type: Bug Bug
    • Status: Closed
    • Priority: Major Major
    • Resolution: Fixed
    • Affects Version/s: 1.8.1
    • Fix Version/s: 1.8.2-RC1, 1.8.2
    • Component/s: ICE-Components
    • Labels:
      None
    • Environment:
      ICEfaces, component-showcase, iPhone

      Description

      File upload from the iPhone causes the thread serving the upload to hang.

      POST /component-showcase/uploadHtml HTTP/1.1
      Host: 10.18.39.60:8080
      User-Agent: Mozilla/5.0 (iPhone; U; CPU iPhone OS 3_0 like Mac OS X; en-us) AppleWebKit/528.18 (KHTML, like Gecko) Version/4.0 Mobile/7A341 Safari/528.16
      Content-Type: multipart/form-data; boundary=----WebKitFormBoundaryKRlAnlJzbOtqlLwK
      Referer: http://10.18.39.60:8080/component-showcase/block/resource/LTI5MDYxNDExOA==/
      Accept: application/xml,application/xhtml+xml,text/html;q=0.9,text/plain;q=0.8,image/png,*/*;q=0.5
      Origin: http://10.18.39.60:8080
      Accept-Language: en-us
      Accept-Encoding: gzip, deflate
      Cookie: JSESSIONID=07BE139C9D1813B44869DD0EB5CDBE1F; bconn=VaJqZkV_qhvmGY44a0-qUg:1:acquired; ice.lease=1247599456150; ice.sessions=VaJqZkV_qhvmGY44a0-qUg#1; updates=
      Content-Length: 276
      Connection: keep-alive

      ------WebKitFormBoundaryKRlAnlJzbOtqlLwK
      Content-Disposition: form-data; name="ice.component"

      iceform:icepnltabset:0:inputFileName
      ------WebKitFormBoundaryKRlAnlJzbOtqlLwK
      Content-Disposition: form-data; name="ice.view"

      1
      ------WebKitFormBoundaryKRlAnlJzbOtqlLwK--

        Activity

        Hide
        Ted Goddard added a comment -

        Likely an edge case in the upload server we should account for; however, ICEfaces 2.0 will probably use a completely new file upload mechanism, so it should simply be retested.

        Show
        Ted Goddard added a comment - Likely an edge case in the upload server we should account for; however, ICEfaces 2.0 will probably use a completely new file upload mechanism, so it should simply be retested.
        Hide
        Mark Collette added a comment -

        But it works from Safari on a desktop computer?

        Show
        Mark Collette added a comment - But it works from Safari on a desktop computer?
        Hide
        Ted Goddard added a comment -

        That's correct. From the desktop, the multiple parts actually contains a file:

        POST /component-showcase/uploadHtml HTTP/1.1
        Host: localhost:8080
        User-Agent: Mozilla/5.0 (Macintosh; U; Intel Mac OS X 10_5_7; en-us) AppleWebKit/530.19.2 (KHTML, like Gecko) Version/4.0.2 Safari/530.19
        Content-Type: multipart/form-data; boundary=----WebKitFormBoundaryIKBO3A4GknoBzt3x
        Referer: http://localhost:8080/component-showcase/block/resource/MTM5NTI4NTgxNw==/
        Accept: application/xml,application/xhtml+xml,text/html;q=0.9,text/plain;q=0.8,image/png,/;q=0.5
        Origin: http://localhost:8080
        Accept-Language: en-us
        Accept-Encoding: gzip, deflate
        Cookie: com.sun.faces.context.flash.PostbackRequest=119; JSESSIONID=86BA66DBFB02A284C1B371CAB1AE655E; bconn=ddEDQIGPaGK3vGrYc0wHCw:2:acquired; ice.connection.lease=1247522019792; ice.lease=1247600459461; ice.sessions=ddEDQIGPaGK3vGrYc0wHCw#1; ice.updated.views=; ice.views=; updates=
        Content-Length: 382
        Connection: keep-alive

        ------WebKitFormBoundaryIKBO3A4GknoBzt3x
        Content-Disposition: form-data; name="ice.component"

        iceform:icepnltabset:0:inputFileName
        ------WebKitFormBoundaryIKBO3A4GknoBzt3x
        Content-Disposition: form-data; name="ice.view"

        2
        ------WebKitFormBoundaryIKBO3A4GknoBzt3x
        Content-Disposition: form-data; name="upload"; filename=""

        -----WebKitFormBoundaryIKBO3A4GknoBzt3x-

        Show
        Ted Goddard added a comment - That's correct. From the desktop, the multiple parts actually contains a file: POST /component-showcase/uploadHtml HTTP/1.1 Host: localhost:8080 User-Agent: Mozilla/5.0 (Macintosh; U; Intel Mac OS X 10_5_7; en-us) AppleWebKit/530.19.2 (KHTML, like Gecko) Version/4.0.2 Safari/530.19 Content-Type: multipart/form-data; boundary=----WebKitFormBoundaryIKBO3A4GknoBzt3x Referer: http://localhost:8080/component-showcase/block/resource/MTM5NTI4NTgxNw==/ Accept: application/xml,application/xhtml+xml,text/html;q=0.9,text/plain;q=0.8,image/png, / ;q=0.5 Origin: http://localhost:8080 Accept-Language: en-us Accept-Encoding: gzip, deflate Cookie: com.sun.faces.context.flash.PostbackRequest=119; JSESSIONID=86BA66DBFB02A284C1B371CAB1AE655E; bconn=ddEDQIGPaGK3vGrYc0wHCw:2:acquired; ice.connection.lease=1247522019792; ice.lease=1247600459461; ice.sessions=ddEDQIGPaGK3vGrYc0wHCw#1; ice.updated.views=; ice.views=; updates= Content-Length: 382 Connection: keep-alive ------WebKitFormBoundaryIKBO3A4GknoBzt3x Content-Disposition: form-data; name="ice.component" iceform:icepnltabset:0:inputFileName ------WebKitFormBoundaryIKBO3A4GknoBzt3x Content-Disposition: form-data; name="ice.view" 2 ------WebKitFormBoundaryIKBO3A4GknoBzt3x Content-Disposition: form-data; name="upload"; filename="" ----- WebKitFormBoundaryIKBO3A4GknoBzt3x -
        Hide
        Ted Goddard added a comment -

        Threads no longer appear to hang since the thread blocking refactoring. The only problem now is that the upload button is not replaced in the iframe when the upload is performed. This indicates an edge case that the Servlet is not handling correctly. However, since the iPhone cannot upload files anyway, fixing this is a low priority.

        Before closing this bug, I recommend that we wait to observe the trunk deployed to component-showcase.icefaces.org.

        Show
        Ted Goddard added a comment - Threads no longer appear to hang since the thread blocking refactoring. The only problem now is that the upload button is not replaced in the iframe when the upload is performed. This indicates an edge case that the Servlet is not handling correctly. However, since the iPhone cannot upload files anyway, fixing this is a low priority. Before closing this bug, I recommend that we wait to observe the trunk deployed to component-showcase.icefaces.org.
        Hide
        Ken Fyten added a comment -

        Verified on iphone 3G with latest trunk.

        Show
        Ken Fyten added a comment - Verified on iphone 3G with latest trunk.

          People

          • Assignee:
            Mark Collette
            Reporter:
            Ted Goddard
          • Votes:
            1 Vote for this issue
            Watchers:
            1 Start watching this issue

            Dates

            • Created:
              Updated:
              Resolved: