diff options
| author |  Daenney <daenney@users.noreply.github.com> | 2023-02-17 12:02:29 +0100 |
|---|---|---|
| committer |  GitHub <noreply@github.com> | 2023-02-17 12:02:29 +0100 |
| commit | 68e6d08c768b789987a753d42f66caf73ce10ee1 (patch) | |
| tree | 1c9eb6da6c326266d653de80684c3aec58922638 /internal/db/bundb/status.go | |
| parent | [bugfix] Set 'discoverable' properly on API accounts (#1511) (diff) | |
| download | gotosocial-68e6d08c768b789987a753d42f66caf73ce10ee1.tar.xz | |
[feature] Add a request ID and include it in logs (#1476)
This adds a lightweight form of tracing to GTS. Each incoming request is
assigned a Request ID which we then pass on and log in all our log
lines. Any function that gets called downstream from an HTTP handler
should now emit a requestID=value pair whenever it logs something.
Co-authored-by: kim <grufwub@gmail.com>
Diffstat (limited to 'internal/db/bundb/status.go')
| -rw-r--r-- | internal/db/bundb/status.go | 10 |
1 files changed, 5 insertions, 5 deletions
diff --git a/internal/db/bundb/status.go b/internal/db/bundb/status.go index 709105f72..2bec07759 100644 --- a/internal/db/bundb/status.go +++ b/internal/db/bundb/status.go @@ -74,7 +74,7 @@ func (s *statusDB) GetStatuses(ctx context.Context, ids []string) ([]*gtsmodel.S // Attempt fetch from DB status, err := s.GetStatusByID(ctx, id) if err != nil { - log.Errorf("GetStatuses: error getting status %q: %v", id, err) + log.Errorf(ctx, "error getting status %q: %v", id, err) continue } @@ -387,7 +387,7 @@ func (s *statusDB) GetStatusChildren(ctx context.Context, status *gtsmodel.Statu // only append children, not the overall parent status entry, ok := e.Value.(*gtsmodel.Status) if !ok { - log.Panic("GetStatusChildren: found status could not be asserted to *gtsmodel.Status") + log.Panic(ctx, "found status could not be asserted to *gtsmodel.Status") } if entry.ID != status.ID { @@ -412,7 +412,7 @@ func (s *statusDB) statusChildren(ctx context.Context, status *gtsmodel.Status, if err := q.Scan(ctx, &childIDs); err != nil { if err != sql.ErrNoRows { - log.Errorf("statusChildren: error getting children for %q: %v", status.ID, err) + log.Errorf(ctx, "error getting children for %q: %v", status.ID, err) } return } @@ -421,7 +421,7 @@ func (s *statusDB) statusChildren(ctx context.Context, status *gtsmodel.Status, // Fetch child with ID from database child, err := s.GetStatusByID(ctx, id) if err != nil { - log.Errorf("statusChildren: error getting child status %q: %v", id, err) + log.Errorf(ctx, "error getting child status %q: %v", id, err) continue } @@ -429,7 +429,7 @@ func (s *statusDB) statusChildren(ctx context.Context, status *gtsmodel.Status, for e := foundStatuses.Front(); e != nil; e = e.Next() { entry, ok := e.Value.(*gtsmodel.Status) if !ok { - log.Panic("statusChildren: found status could not be asserted to *gtsmodel.Status") + log.Panic(ctx, "found status could not be asserted to *gtsmodel.Status") } if child.InReplyToAccountID != "" && entry.ID == child.InReplyToID { |