Added fix to retrieve correct s3 urls (#2465)

* Added fix to retrieve correct s3 urls

* Fixes to S3 url

* Squashing commits

* Fixing if conditions for storagetype in geturl

pkg/storagesvc/storagesvc.go

@@ -231,19 +231,23 @@ func (ss *StorageService) downloadHandler(w http.ResponseWriter, r *http.Request
 
 func (ss *StorageService) infoHandler(w http.ResponseWriter, r *http.Request) {
 
-	fileId, err := ss.getIdFromRequest(r)
+	fileID, err := ss.getIdFromRequest(r)
 	if err != nil {
 		http.Error(w, err.Error(), http.StatusBadRequest)
 		return
 	}
 
-	itemURL, err := ss.storageClient.getURL(fileId)
+	_, err = ss.storageClient.container.Item(fileID)
 	if err != nil {
-		http.Error(w, err.Error(), http.StatusBadRequest)
+		http.Error(w, err.Error(), http.StatusNotFound)
+		return
 	}
 
-	w.Header().Add("X-FISSION-ARCHIVEURL", itemURL.String())
-
+	storageType := ss.storageClient.config.storage.getStorageType()
+	if storageType == StorageTypeS3 {
+		w.Header().Add("X-FISSION-BUCKET", ss.storageClient.config.storage.getContainerName())
+	}
+	w.Header().Add("X-FISSION-STORAGETYPE", string(storageType))
 }
 
 func (ss *StorageService) healthHandler(w http.ResponseWriter, r *http.Request) {

pkg/storagesvc/stowClient.go

@@ -20,7 +20,6 @@ import (
 	"fmt"
 	"io"
 	"mime/multipart"
-	"net/url"
 	"os"
 	"strings"
 	"time"
@@ -183,18 +182,6 @@ func (client *StowClient) removeFileByID(itemID string) error {
 	return client.container.RemoveItem(itemID)
 }
 
-func (client *StowClient) getURL(itemID string) (*url.URL, error) {
-	item, err := client.container.Item(itemID)
-	if err != nil {
-		if err == stow.ErrNotFound {
-			return nil, ErrNotFound
-		} else {
-			return nil, ErrRetrievingItem
-		}
-	}
-	return item.URL(), nil
-}
-
 func (client *StowClient) getFileSize(itemID string) (int64, error) {
 	item, err := client.container.Item(itemID)
 	if err != nil {