checker: Add NewLogger constructor for DetailLogger impl

Signed-off-by: Stephen Augustus <[email protected]>
ossf · Feb 16, 2022 · c80a8f2 · c80a8f2
1 parent ba503c3
commit c80a8f2
Show file tree

Hide file tree

Showing 2 changed files with 6 additions and 3 deletions.
diff --git a/checker/check_runner.go b/checker/check_runner.go
@@ -79,12 +79,11 @@ func (r *Runner) Run(ctx context.Context, c Check) CheckResult {
 	startTime := time.Now()
 
 	var res CheckResult
-	var l logger
+	l := NewLogger()
 	for retriesRemaining := checkRetries; retriesRemaining > 0; retriesRemaining-- {
 		checkRequest := r.CheckRequest
 		checkRequest.Ctx = ctx
-		l = logger{}
-		checkRequest.Dlogger = &l
+		checkRequest.Dlogger = l
 		res = c.Fn(&checkRequest)
 		if res.Error2 != nil && errors.Is(res.Error2, sce.ErrRepoUnreachable) {
 			checkRequest.Dlogger.Warn(&LogMessage{

diff --git a/checker/detail_logger_impl.go b/checker/detail_logger_impl.go
@@ -18,6 +18,10 @@ type logger struct {
 	logs []CheckDetail
 }
 
+func NewLogger() *logger {
+	return &logger{}
+}
+
 func (l *logger) Info(msg *LogMessage) {
 	cd := CheckDetail{
 		Type: DetailInfo,