TrueCloudLab/xk6-frostfs
39
0
Fork 17
Code Issues 36 Pull requests 2 Releases Activity Actions

[#140] native,s3: Do not set Success flag on hash mismatch
All checks were successful
DCO action / DCO (pull_request) Successful in 1m4s
Details
Tests and linters / Tests (1.21) (pull_request) Successful in 2m0s
Details
Tests and linters / Tests (1.22) (pull_request) Successful in 2m19s
Details
Tests and linters / Lint (pull_request) Successful in 3m32s
Details
Tests and linters / Tests with -race (pull_request) Successful in 3m52s
Details

Browse source 
Signed-off-by: Evgenii Stratonikov <e.stratonikov@yadro.com>
This commit is contained in:
Evgenii Stratonikov 2024-05-30 13:46:25 +03:00
parent 31fac75743
commit de61aef66e
2 changed files with 2 additions and 2 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

2
internal/native/client.go
View file

@ -240,7 +240,7 @@ func (c *Client) VerifyHash(containerID, objectID, expectedHash string) VerifyHa
}
actualHash := hex.EncodeToString(hasher.Sum(nil))
if actualHash != expectedHash {
return VerifyHashResponse{Success: true, Error: "hash mismatch"}
return VerifyHashResponse{Success: false, Error: "hash mismatch"}
}
return VerifyHashResponse{Success: true}

2
internal/s3/client.go
View file

@ -180,7 +180,7 @@ func (c *Client) VerifyHash(bucket, key, expectedHash string) VerifyHashResponse
}
actualHash := hex.EncodeToString(hasher.Sum(nil))
if actualHash != expectedHash {
return VerifyHashResponse{Success: true, Error: "hash mismatch"}
return VerifyHashResponse{Success: false, Error: "hash mismatch"}
}
return VerifyHashResponse{Success: true}