fixes following review

armosec · Nov 21, 2023 · 972e002 · 972e002
1 parent 005f3f2
commit 972e002
Show file tree

Hide file tree

Showing 2 changed files with 2 additions and 5 deletions.
diff --git a/.github/workflows/pr.yaml b/.github/workflows/pr.yaml
@@ -26,9 +26,6 @@ jobs:
 
     - name: Get dependencies
       run: go mod download
-
-    - name: Build
-      run: go build -v ./...
 
     - name: Test
-      run: go test ./... -v --race
+      run: go test ./... -v --race -cover
diff --git a/s3connector/s3.go b/s3connector/s3.go
@@ -117,7 +117,7 @@ func (s *s3ObjectStorage) GetObject(objPath S3ObjectPath) (io.ReadCloser, error)
 
 	if objPath.Range != nil {
 		if objPath.Range.Start < 0 || objPath.Range.End <= objPath.Range.Start {
-			return nil, fmt.Errorf("invalid range: start must be non-negative and end must be greater than start")
+			return nil, fmt.Errorf("invalid range: start must be non-negative and end must be greater than start, ranges are: %v", objPath.Range)
 		}
 		objRange = aws.String(fmt.Sprintf("bytes=%d-%d", objPath.Range.Start, objPath.Range.End))
 	}