Correct handleAllDbs audit fields

base/audit_events.go

@@ -431,8 +431,10 @@ var AuditEvents = events{
 	AuditIDDatabaseAllRead: {
 		Name:        "Read all databases",
 		Description: "All databases were viewed",
+		MandatoryFields: AuditFields{
+			AuditFieldDBNames: []string{"list", "of", "db", "names"},
+		},
 		mandatoryFieldGroups: []fieldGroup{
-			fieldGroupDatabase,
 			fieldGroupRequest,
 			// fieldGroupAuthenticated, // FIXME: CBG-3973,
 		},

base/audit_events_fields.go

@@ -33,6 +33,7 @@ const (
 	AuditEffectiveUserID = "effective_userid"
 	AuditFieldAuditScope = "audit_scope"
 	AuditFieldFileName   = "filename"
+	AuditFieldDBNames    = "db_names"
 
 	// AuditIDSyncGatewayStartup AuditID = 53260
 	AuditFieldSGVersion                      = "sg_version"

rest/api.go

@@ -80,12 +80,19 @@ func (h *handler) handlePing() error {
 }
 
 func (h *handler) handleAllDbs() error {
-	base.Audit(h.ctx(), base.AuditIDDatabaseAllRead, nil)
-	if h.getBoolQuery("verbose") {
-		h.writeJSON(h.server.allDatabaseSummaries())
-		return nil
+	verbose := h.getBoolQuery("verbose")
+	var dbNames []string
+	if verbose {
+		summaries := h.server.allDatabaseSummaries()
+		for _, summary := range summaries {
+			dbNames = append(dbNames, summary.DBName)
+		}
+		h.writeJSON(summaries)
+	} else {
+		dbNames = h.server.AllDatabaseNames()
+		h.writeJSON(dbNames)
 	}
-	h.writeJSON(h.server.AllDatabaseNames())
+	base.Audit(h.ctx(), base.AuditIDDatabaseAllRead, base.AuditFields{base.AuditFieldDBNames: dbNames, "verbose": verbose})
 	return nil
 }