extern/gotosocial
1
1
Fork 0
mirror of https://github.com/superseriousbusiness/gotosocial.git synced 2025-02-09 23:22:24 +01:00
Code Issues Packages Projects Releases Wiki Activity

Always load target account settings

Browse Source
This commit is contained in:
Vyr Cossont 2025-01-21 17:19:28 -08:00
parent b982944e85
commit a8ae0e3173
1 changed files with 10 additions and 3 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

13
internal/webpush/realsender.go
View File

@ -85,7 +85,7 @@ func(subscription *gtsmodel.WebPushSubscription) bool {
return nil
}
// Load VAPID keys into webpush-go options struct.
// Get VAPID keys.
vapidKeyPair, err := r.state.DB.GetVAPIDKeyPair(ctx)
if err != nil {
return gtserror.Newf("error getting VAPID key pair: %w", err)
@ -103,8 +103,13 @@ func(subscription *gtsmodel.WebPushSubscription) bool {
vapidSubjectEmail = "admin@" + domain
}
// Get target account settings.
targetAccountSettings, err := r.state.DB.GetAccountSettings(ctx, notification.TargetAccountID)
if err != nil {
return gtserror.Newf("error getting settings for account %s: %w", notification.TargetAccountID, err)
}
// Get API representations of notification and accounts involved.
// This also loads the target account's settings.
apiNotification, err := r.tc.NotificationToAPINotification(ctx, notification, filters, mutes)
if err != nil {
return gtserror.Newf("error converting notification %s to API representation: %w", notification.ID, err)
@ -117,6 +122,7 @@ func(subscription *gtsmodel.WebPushSubscription) bool {
ctx,
vapidKeyPair,
vapidSubjectEmail,
targetAccountSettings,
subscription,
notification,
apiNotification,
@ -139,6 +145,7 @@ func (r *realSender) sendToSubscription(
ctx context.Context,
vapidKeyPair *gtsmodel.VAPIDKeyPair,
vapidSubjectEmail string,
targetAccountSettings *gtsmodel.AccountSettings,
subscription *gtsmodel.WebPushSubscription,
notification *gtsmodel.Notification,
apiNotification *apimodel.Notification,
@ -165,7 +172,7 @@ func (r *realSender) sendToSubscription(
Title: formatNotificationTitle(ctx, subscription, notification, apiNotification),
Body: formatNotificationBody(apiNotification),
Icon: apiNotification.Account.Avatar,
PreferredLocale: notification.TargetAccount.Settings.Language,
PreferredLocale: targetAccountSettings.Language,
AccessToken: token.Access,
}