Sling
  1. Sling
  2. SLING-286

Use ServletContext to resolve content type of uploaded file

    Details

    • Type: Bug Bug
    • Status: Closed
    • Priority: Major Major
    • Resolution: Fixed
    • Affects Version/s: None
    • Fix Version/s: 3
    • Component/s: Servlets
    • Labels:
      None

      Description

      The ujax file upload support currently uses the MimeTypeService to find out the content type of an uploaded file if no such type is explicitly set. This creates a dependency on the MimeTypeService, requires checking for service availability and more. Better would be for the file upload functionality to use the ServletContext of the UjaxPostServlet to resolve the content type.

      In fact, the ServletContext of the UjaxPostServlet uses the MimeTypeService for content type resolution, so we still use the full Sling functionality of content type resolution.

        Activity

        Felix Meschberger created issue -
        Felix Meschberger made changes -
        Field Original Value New Value
        Component/s ujax [ 12312181 ]
        Felix Meschberger made changes -
        Status Open [ 1 ] In Progress [ 3 ]
        Felix Meschberger made changes -
        Status In Progress [ 3 ] Open [ 1 ]
        Carsten Ziegeler made changes -
        Resolution Fixed [ 1 ]
        Status Open [ 1 ] Closed [ 6 ]
        Felix Meschberger made changes -
        Workflow jira [ 12424628 ] no-reopen-closed,doc-test-required [ 12475519 ]
        Felix Meschberger made changes -
        Component/s Servlets [ 12313028 ]
        Component/s Servlets Post [ 12312181 ]
        Gavin made changes -
        Workflow no-reopen-closed,doc-test-required [ 12475519 ] Copy of no-reopen-closed,doc-test-required [ 12763615 ]
        Gavin made changes -
        Workflow Copy of no-reopen-closed,doc-test-required [ 12763615 ] no-reopen-closed,doc-test-required [ 12765592 ]
        Gavin made changes -
        Workflow no-reopen-closed,doc-test-required [ 12765592 ] re-open possible,doc-test-required [ 12788888 ]
        Gavin made changes -
        Workflow re-open possible,doc-test-required [ 12788888 ] no-reopen-closed,doc-test-required [ 12793075 ]

          People

          • Assignee:
            Felix Meschberger
            Reporter:
            Felix Meschberger
          • Votes:
            0 Vote for this issue
            Watchers:
            0 Start watching this issue

            Dates

            • Created:
              Updated:
              Resolved:

              Development