packager.init: Do not include maintainer if user name empty

chris-crone · chris-crone · commit 019f459edaf6 · 2018-08-29T13:55:43.000+02:00
Signed-off-by: Christopher Crone &lt;christopher.crone@docker.com&gt;
diff --git a/internal/packager/init.go b/internal/packager/init.go
@@ -237,12 +237,10 @@ func newMetadata(name string, description string, maintainers []string) metadata
 		Description: description,
 	}
 	if len(maintainers) == 0 {
-		var userName string
 		userData, _ := user.Current()
-		if userData != nil {
-			userName = userData.Username
+		if userData != nil && userData.Username != "" {
+			res.Maintainers = []metadata.Maintainer{{Name: userData.Username}}
 		}
-		res.Maintainers = []metadata.Maintainer{{Name: userName}}
 	} else {
 		res.Maintainers = parseMaintainersData(maintainers)
 	}

Original file line number	Diff line number	Diff line change
`@@ -237,12 +237,10 @@ func newMetadata(name string, description string, maintainers []string) metadata`
`237`	`237`	`Description: description,`
`238`	`238`	`}`
`239`	`239`	`if len(maintainers) == 0 {`
`240`		`- var userName string`
`241`	`240`	`userData, _ := user.Current()`
`242`		`- if userData != nil {`
`243`		`- userName = userData.Username`
	`241`	`+ if userData != nil && userData.Username != "" {`
	`242`	`+ res.Maintainers = []metadata.Maintainer{{Name: userData.Username}}`
`244`	`243`	`}`
`245`		`- res.Maintainers = []metadata.Maintainer{{Name: userName}}`
`246`	`244`	`} else {`
`247`	`245`	`res.Maintainers = parseMaintainersData(maintainers)`
`248`	`246`	`}`