Uploaded image for project: 'Jenkins'
  1. Jenkins
  2. JENKINS-50772

Content type not set reasonably

    XMLWordPrintable

    Details

    • Similar Issues:

      Description

      It seems that when you upload even a file named something.txt, it gets an octet-stream content type on S3, so even if you click the view link in Classic UI on the artifacts page, your browser gets a downloaded file rather than displaying inline. Must be some way to tell jClouds/S3 to sniff content type automatically. OTOH there may be some dangerous types which we would want to serve only as plain text or binary.

        Attachments

          Issue Links

            Activity

            Hide
            jglick Jesse Glick added a comment -

            MutableContentMetadata.setContentType? But Jenkins has no API for detecting content type of files.

            Ideally VirtualFile.toExternalURL would allow you to pass a “view mode” parameter requesting a content type…if S3 supports that.

            Show
            jglick Jesse Glick added a comment - MutableContentMetadata.setContentType ? But Jenkins has no API for detecting content type of files. Ideally VirtualFile.toExternalURL would allow you to pass a “view mode” parameter requesting a content type…if S3 supports that.
            Hide
            akdor1154 Jarrad Whitaker added a comment -

            RobustHttpClient.uploadFile also has no way of setting a content type, which is another blocker for this I think?

            Show
            akdor1154 Jarrad Whitaker added a comment - RobustHttpClient.uploadFile also has no way of setting a content type, which is another blocker for this I think?
            Hide
            akdor1154 Jarrad Whitaker added a comment - - edited

            (this is actually a blocker for our team to use this plugin, "I can't see my test report when I click the link ")

            Show
            akdor1154 Jarrad Whitaker added a comment - - edited (this is actually a blocker for our team to use this plugin, "I can't see my test report when I click the link ")
            Hide
            jglick Jesse Glick added a comment -

            I doubt RobustHttpClient is involved; the content type setting would be done on the master side using the AWS SDK. I think. Will not know for sure until a fix is implemented. Perhaps David Currie wants to take a stab at it?

            Show
            jglick Jesse Glick added a comment - I doubt RobustHttpClient is involved; the content type setting would be done on the master side using the AWS SDK. I think. Will not know for sure until a fix is implemented. Perhaps David Currie wants to take a stab at it?
            Show
            akdor1154 Jarrad Whitaker added a comment - - edited Unless I'm reading wrong, by https://github.com/jenkinsci/artifact-manager-s3-plugin/blob/master/src/main/java/io/jenkins/plugins/artifact_manager_jclouds/JCloudsArtifactManager.java#L115 and https://github.com/jenkinsci/artifact-manager-s3-plugin/blob/master/src/main/java/io/jenkins/plugins/artifact_manager_jclouds/JCloudsArtifactManager.java#L134, it works by getting a presigned S3 url and `uploadFile`ing to it with RobustHttpClient.
            Hide
            dcurrie David Currie added a comment -

            My reading is the same as Jarrad Whitaker. The S3 API indicates that the default Content-Type on a PUT is binary/octet-stream unless you specify otherwise.

            Show
            dcurrie David Currie added a comment - My reading is the same as Jarrad Whitaker . The S3 API indicates that the default Content-Type on a PUT is binary/octet-stream unless you specify otherwise.
            Hide
            jglick Jesse Glick added a comment -

            Right, the question is whether the URL itself can encode the desired content type or whether this needs to be handled as an HTTP header by the uploading code. In the latter case, RobustHttpClient.uploadFile would need to be amended to set a header.

            (Then the next question is what content type to pick. Files.probeContentType seems like the right choice.)

            Show
            jglick Jesse Glick added a comment - Right, the question is whether the URL itself can encode the desired content type or whether this needs to be handled as an HTTP header by the uploading code. In the latter case, RobustHttpClient.uploadFile would need to be amended to set a header. (Then the next question is what content type to pick. Files.probeContentType seems like the right choice.)
            Hide
            nanonyme Seppo Yli-Olli added a comment - - edited

            Even if you only populated content type for most common things (txt, html), this would still be useful. Is this being actively worked on? This limitation makes using HTML reports through Jenkins web UI quite painful if this plugin is active. My assumption based on what I've seen thus far is latter option is the right one but it could be that I'm just missing some details of presigned URL's.

            Show
            nanonyme Seppo Yli-Olli added a comment - - edited Even if you only populated content type for most common things (txt, html), this would still be useful. Is this being actively worked on? This limitation makes using HTML reports through Jenkins web UI quite painful if this plugin is active. My assumption based on what I've seen thus far is latter option is the right one but it could be that I'm just missing some details of presigned URL's.

              People

              • Assignee:
                Unassigned
                Reporter:
                jglick Jesse Glick
              • Votes:
                5 Vote for this issue
                Watchers:
                6 Start watching this issue

                Dates

                • Created:
                  Updated: