diff options
| author |  kim <89579420+NyaaaWhatsUpDoc@users.noreply.github.com> | 2024-06-06 08:50:14 +0000 |
|---|---|---|
| committer |  GitHub <noreply@github.com> | 2024-06-06 10:50:14 +0200 |
| commit | 3b7faac604000297b74baf8f922c79c6b387217d (patch) | |
| tree | 05b1d32eeaa902571a9ec1fb0d643a08b9be2a86 /internal/federation/dereferencing/status.go | |
| parent | [chore] Fiddle with CI tests; use wasmsqlite3 for CI tests (#2966) (diff) | |
| download | gotosocial-3b7faac604000297b74baf8f922c79c6b387217d.tar.xz | |
[bugfix] concurrent map writes in dereferencer media processing maps (#2964)
* removes the avatar / header deref maps as we now have per-uri status / account locks, adds retries on data-races, adds separate emoji map mutex
* work with a copy of account / status for each retry loop
* revert to old data race behaviour, it gets too complicated otherwise
---------
Co-authored-by: tobi <tobi.smethurst@protonmail.com>
Diffstat (limited to 'internal/federation/dereferencing/status.go')
| -rw-r--r-- | internal/federation/dereferencing/status.go | 4 |
1 files changed, 2 insertions, 2 deletions
diff --git a/internal/federation/dereferencing/status.go b/internal/federation/dereferencing/status.go index bd50a08fd..69627adc2 100644 --- a/internal/federation/dereferencing/status.go +++ b/internal/federation/dereferencing/status.go @@ -285,7 +285,7 @@ func (d *Dereferencer) enrichStatusSafely( requestUser string, uri *url.URL, status *gtsmodel.Status, - apubStatus ap.Statusable, + statusable ap.Statusable, ) (*gtsmodel.Status, ap.Statusable, bool, error) { uriStr := status.URI @@ -313,7 +313,7 @@ func (d *Dereferencer) enrichStatusSafely( requestUser, uri, status, - apubStatus, + statusable, ) if gtserror.StatusCode(err) >= 400 { |