Quote string arguments in errors

PiperOrigin-RevId: 728627723
google · Feb 19, 2025 · cc5de23 · cc5de23
1 parent d683505
commit cc5de23
Show file tree

Hide file tree

Showing 3 changed files with 5 additions and 5 deletions.
diff --git a/detector/list/list.go b/detector/list/list.go
@@ -114,7 +114,7 @@ func DetectorsFromNames(names []string) ([]detector.Detector, error) {
 				}
 			}
 		} else {
-			return nil, fmt.Errorf("unknown detector %s", n)
+			return nil, fmt.Errorf("unknown detector %q", n)
 		}
 	}
 	result := make([]detector.Detector, 0, len(resultMap))

diff --git a/extractor/filesystem/list/list.go b/extractor/filesystem/list/list.go
@@ -289,7 +289,7 @@ func ExtractorsFromNames(names []string) ([]filesystem.Extractor, error) {
 				}
 			}
 		} else {
-			return nil, fmt.Errorf("unknown extractor %s", n)
+			return nil, fmt.Errorf("unknown extractor %q", n)
 		}
 	}
 	result := make([]filesystem.Extractor, 0, len(resultMap))
@@ -303,7 +303,7 @@ func ExtractorsFromNames(names []string) ([]filesystem.Extractor, error) {
 func ExtractorFromName(name string) (filesystem.Extractor, error) {
 	es, ok := extractorNames[strings.ToLower(name)]
 	if !ok {
-		return nil, fmt.Errorf("unknown extractor %s", name)
+		return nil, fmt.Errorf("unknown extractor %q", name)
 	}
 	if len(es) != 1 || es[0].Name() != name {
 		return nil, fmt.Errorf("not an exact name for an extractor: %s", name)

diff --git a/extractor/standalone/list/list.go b/extractor/standalone/list/list.go
@@ -113,7 +113,7 @@ func FilterByCapabilities(exs []standalone.Extractor, capabs *plugin.Capabilitie
 func ExtractorFromName(name string) (standalone.Extractor, error) {
 	es, ok := extractorNames[strings.ToLower(name)]
 	if !ok {
-		return nil, fmt.Errorf("unknown extractor %s", name)
+		return nil, fmt.Errorf("unknown extractor %q", name)
 	}
 	if len(es) != 1 || es[0].Name() != name {
 		return nil, fmt.Errorf("not an exact name for an extractor: %s", name)
@@ -132,7 +132,7 @@ func ExtractorsFromNames(names []string) ([]standalone.Extractor, error) {
 				}
 			}
 		} else {
-			return nil, fmt.Errorf("unknown extractor %s", n)
+			return nil, fmt.Errorf("unknown extractor %q", n)
 		}
 	}
 	result := make([]standalone.Extractor, 0, len(resultMap))