We send push notifications to all known user subscription endpoints. These can be removed by the user or outdated after some time. We are then sending notifications to endpoints that no longer exist.

The endpoint will report back if it was send successfully or not, so we can just remove the subscription endpoints who report a failure.

This could save a lot of time and it helps clean the database.

Comments

bramtenhove created an issue. See original summary.

bramtenhove’s picture

bramtenhove CreditAttribution: bramtenhove for Open Social commented
Status: Active » Needs review

Pull request created here: https://github.com/goalgorilla/social_pwa/pull/8

  • bramtenhove committed 015267f on 8.x-1.x 
    Issue #3059964 by bramtenhove - Remove unneeded class
  • ronaldtebrake authored 18942a5 on 8.x-1.x
    Merge pull request #8 from goalgorilla/feature/3059964-clean-up-...
  • bramtenhove committed d71da82 on 8.x-1.x
    Issue #3059964 by bramtenhove - Clean up subscription endpoints that are...
ronaldtebrake’s picture

ronaldtebrake CreditAttribution: ronaldtebrake as a volunteer and at Open Social for Open Social commented
Status: Needs review » Fixed

Status: Fixed » Closed (fixed)

Automatically closed - issue fixed for 2 weeks with no activity.