Fixed XenForo tries to send push notifications to invalid URLs

Kirby

Well-known member
Affected version
2.3.7
I havn't found a definitive reference for this yet, but it seems that Chrome / Chromium sometimes generates placeholder URLs starting with https://permanently-removed.invalid/fcm/send for invalid / expired push subscriptions, see https://xenforo.com/community/threads/push-notification-failure-curl-error-6.234166

XenForo currently accepts those URLs and tries to send notifications there which obviously fails and kinda floods the error log if those endpoints are utilized frequently (which happened to us yesterday).

Suggested Fix
Ignore endpoint URLs starting with https://permanently-removed.invalid/ in first place, eg. don't even store them in DB by returning false for such URLs in UserPushRepository::validateSubscriptionDetails
 
Last edited:
Back
Top Bottom