Fix link physical address response to contain full path

treeverse · Oct 10, 2023 · 32177e6 · 32177e6
1 parent c0d5d31
commit 32177e6
Show file tree

Hide file tree

Showing 3 changed files with 3 additions and 6 deletions.
diff --git a/esti/lakectl_test.go b/esti/lakectl_test.go
@@ -111,7 +111,7 @@ func TestLakectlPreSignUpload(t *testing.T) {
 
 	filePath := "ro_1k.1"
 	vars["FILE_PATH"] = filePath
-	RunCmdAndVerifySuccessWithFile(t, Lakectl()+" fs upload -s files/ro_1k lakefs://"+repoName+"/"+mainBranch+"/"+filePath+" --pre-sign", false, "lakectl_fs_upload_pre_signed", vars)
+	RunCmdAndVerifySuccessWithFile(t, Lakectl()+" fs upload -s files/ro_1k lakefs://"+repoName+"/"+mainBranch+"/"+filePath+" --pre-sign", false, "lakectl_fs_upload", vars)
 
 }
 

diff --git a/pkg/api/controller.go b/pkg/api/controller.go
@@ -394,7 +394,7 @@ func (c *Controller) LinkPhysicalAddress(w http.ResponseWriter, r *http.Request,
 		Mtime:           entry.CreationDate.Unix(),
 		Path:            entry.Path,
 		PathType:        entryTypeObject,
-		PhysicalAddress: entry.PhysicalAddress,
+		PhysicalAddress: qk.Format(),
 		SizeBytes:       swag.Int64(entry.Size),
 	}
 

diff --git a/pkg/api/helpers/upload.go b/pkg/api/helpers/upload.go
@@ -178,10 +178,7 @@ func clientUploadPreSignHelper(ctx context.Context, client apigen.ClientWithResp
 		return nil, fmt.Errorf("link object to backing store: %w", err)
 	}
 	if linkResp.JSON200 != nil {
-		objStat := *linkResp.JSON200
-		// this is a workaround for the fact that the API does not return the full physical address
-		objStat.PhysicalAddress = preSignURL
-		return &objStat, nil
+		return linkResp.JSON200, nil
 	}
 	if linkResp.JSON409 != nil {
 		return nil, ErrConflict