fix: log auth actions (#479)

kangmingtay · web-flow · commit e01be0dfbde0 · 2022-06-01T15:28:59.000-07:00
diff --git a/models/audit_log_entry.go b/models/audit_log_entry.go
@@ -8,6 +8,7 @@ import (
 	"github.com/gofrs/uuid"
 	"github.com/netlify/gotrue/storage"
 	"github.com/pkg/errors"
+	"github.com/sirupsen/logrus"
 )
 
 type AuditAction string
@@ -96,7 +97,12 @@ func NewAuditLogEntry(tx *storage.Connection, instanceID uuid.UUID, actor *User,
 		l.Payload["traits"] = traits
 	}
 
-	return errors.Wrap(tx.Create(&l), "Database error creating audit log entry")
+	if err := tx.Create(&l); err != nil {
+		return errors.Wrap(err, "Database error creating audit log entry")
+	}
+
+	logrus.Infof("{\"actor_id\": %v, \"action\": %v, \"timestamp\": %v, \"log_type\": %v}", actor.ID, action, l.Payload["timestamp"], actionLogTypeMap[action])
+	return nil
 }
 
 func FindAuditLogEntries(tx *storage.Connection, instanceID uuid.UUID, filterColumns []string, filterValue string, pageParams *Pagination) ([]*AuditLogEntry, error) {

Original file line number	Diff line number	Diff line change
`@@ -8,6 +8,7 @@ import (`
`8`	`8`	`"github.com/gofrs/uuid"`
`9`	`9`	`"github.com/netlify/gotrue/storage"`
`10`	`10`	`"github.com/pkg/errors"`
	`11`	`+ "github.com/sirupsen/logrus"`
`11`	`12`	`)`
`12`	`13`
`13`	`14`	`type AuditAction string`
`@@ -96,7 +97,12 @@ func NewAuditLogEntry(tx storage.Connection, instanceID uuid.UUID, actor User,`
`96`	`97`	`l.Payload["traits"] = traits`
`97`	`98`	`}`
`98`	`99`
`99`		`- return errors.Wrap(tx.Create(&l), "Database error creating audit log entry")`
	`100`	`+ if err := tx.Create(&l); err != nil {`
	`101`	`+ return errors.Wrap(err, "Database error creating audit log entry")`
	`102`	`+ }`
	`103`	`+`
	`104`	`+ logrus.Infof("{\"actor_id\": %v, \"action\": %v, \"timestamp\": %v, \"log_type\": %v}", actor.ID, action, l.Payload["timestamp"], actionLogTypeMap[action])`
	`105`	`+ return nil`
`100`	`106`	`}`
`101`	`107`
`102`	`108`	`func FindAuditLogEntries(tx storage.Connection, instanceID uuid.UUID, filterColumns []string, filterValue string, pageParams Pagination) ([]*AuditLogEntry, error) {`