Discovery: do not log 'missing credentials' if other DIDs are success…

…fully registered (#3447)
nuts-foundation · Oct 4, 2024 · 9ec13e7 · 9ec13e7
1 parent 1e3e2c1
commit 9ec13e7
Showing 1 changed file with 6 additions and 1 deletion.
diff --git a/discovery/client.go b/discovery/client.go
@@ -99,7 +99,12 @@ func (r *defaultClientRegistrationManager) activate(ctx context.Context, service
 	for _, subjectDID := range subjectDIDs {
 		err := r.registerPresentation(ctx, subjectDID, service, parameters)
 		if err != nil {
-			loopErrs = append(loopErrs, fmt.Errorf("%s: %w", subjectDID.String(), err))
+			if !errors.Is(err, pe.ErrNoCredentials) { // ignore missing credentials
+				loopErrs = append(loopErrs, fmt.Errorf("%s: %w", subjectDID.String(), err))
+			} else {
+				// trace logging for missing credentials
+				log.Logger().Tracef("Missing credentials for Discovery Service (service=%s, subject=%s, did=%s): %s", service.ID, subjectID, subjectDID, err.Error())
+			}
 		} else {
 			registeredDIDs = append(registeredDIDs, subjectDID.String())
 		}