fixup! Refactor to work with the filePersister

grafana · Jan 30, 2024 · 39d2db9 · 39d2db9
1 parent c502205
commit 39d2db9
Showing 1 changed file with 3 additions and 3 deletions.
diff --git a/common/screenshotter.go b/common/screenshotter.go
@@ -65,8 +65,8 @@ type screenshotter struct {
 	persister *storage.LocalFilePersister
 }
 
-func newScreenshotter(ctx context.Context, persister *storage.LocalFilePersister) *screenshotter {
-	return &screenshotter{ctx, persister}
+func newScreenshotter(ctx context.Context, fp *storage.LocalFilePersister) *screenshotter {
+	return &screenshotter{ctx, fp}
 }
 
 func (s *screenshotter) fullPageSize(p *Page) (*Size, error) {
@@ -218,7 +218,7 @@ func (s *screenshotter) screenshot(
 	// Save screenshot capture to file
 	if path != "" {
 		if err := s.persister.Persist(path, bytes.NewBuffer(buf)); err != nil {
-			return nil, fmt.Errorf("saving screenshot")
+			return nil, fmt.Errorf("saving screenshot: %w", err)
 		}
 	}