blob: permit skipping the configuration of the Content-Disposition header (#4832)

This change would allow users to disable the `Content-Disposition`
header that is set for blob storage operations. The application will
continue to set a default value for `Content-Disposition` of
`attachment; filename={{.Filename}}` if no value was provided by the
user. However, with this change, users can now specifically disable this
header by setting the value to "-" in the configuration.

We feel this would be a nice solution for this issue:
https://github.com/Homebrew/brew/issues/15604

internal/pipe/blob/blob_test.go

@@ -75,8 +75,9 @@ func TestDefaults(t *testing.T) {
 				ContentDisposition: "inline",
 			},
 			{
-				Bucket:   "foobar",
-				Provider: "gcs",
+				Bucket:             "foobar",
+				Provider:           "gcs",
+				ContentDisposition: "-",
 			},
 			{
 				Bucket:        "deprecated",
@@ -100,7 +101,7 @@ func TestDefaults(t *testing.T) {
 			Bucket:             "foobar",
 			Provider:           "gcs",
 			Directory:          "{{ .ProjectName }}/{{ .Tag }}",
-			ContentDisposition: "attachment;filename={{.Filename}}",
+			ContentDisposition: "",
 		},
 		{
 			Bucket:             "deprecated",