[chore/bugfix] Demote failed inbox forwarding to warn log rather than error return (#1892)

This commit is contained in:
tobi 2023-06-14 15:08:31 +02:00 committed by tsmethurst
parent dca7a81d80
commit ea22819c7f

View file

@ -231,8 +231,9 @@ func (f *federatingActor) PostInboxScheme(ctx context.Context, w http.ResponseWr
// is updated, and/or federating callbacks are handled // is updated, and/or federating callbacks are handled
// properly. // properly.
if !errors.Is(err, db.ErrAlreadyExists) { if !errors.Is(err, db.ErrAlreadyExists) {
err = fmt.Errorf("PostInboxScheme: error calling sideEffectActor.InboxForwarding: %w", err) // Failed inbox forwarding is not a show-stopper,
return false, gtserror.NewErrorInternalError(err) // and doesn't even necessarily denote a real error.
l.Warnf("error calling sideEffectActor.InboxForwarding: %q", err)
} }
} }