nixcache: only print warning if URI isn't empty (#1974)

gcurtis · web-flow · commit dc3ea965ec34 · 2024-04-08T18:05:14.000-04:00
diff --git a/internal/devbox/providers/nixcache/nixcache.go b/internal/devbox/providers/nixcache/nixcache.go
@@ -205,7 +205,9 @@ func (p *Provider) URI(ctx context.Context) (string, error) {
 	if err != nil {
 		return "", redact.Errorf("nixcache: get uri: %w", redact.Safe(err))
 	}
-	checkIfUserCanAddSubstituter(ctx)
+	if uri != "" {
+		checkIfUserCanAddSubstituter(ctx)
+	}
 	return uri, nil
 }
 

Original file line number	Diff line number	Diff line change
`@@ -205,7 +205,9 @@ func (p *Provider) URI(ctx context.Context) (string, error) {`
`205`	`205`	`if err != nil {`
`206`	`206`	`return "", redact.Errorf("nixcache: get uri: %w", redact.Safe(err))`
`207`	`207`	`}`
`208`		`- checkIfUserCanAddSubstituter(ctx)`
	`208`	`+ if uri != "" {`
	`209`	`+ checkIfUserCanAddSubstituter(ctx)`
	`210`	`+ }`
`209`	`211`	`return uri, nil`
`210`	`212`	`}`
`211`	`213`