django/djangoproject.com

fundraising: 'customer.subscription.deleted' webhook event always gets 404 response

Open

#764 ouverte le 4 mai 2017

Voir sur GitHub
 (5 commentaires) (0 réactions) (0 assignés)Python (1 100 forks)user submission
bugfundraisinggood first issue

Métriques du dépôt

Stars
 (2 001 stars)
Métriques de merge PR
 (Merge moyen 7j 20h) (25 PRs mergées en 30 j)

Description

The response to stripe 'customer.subscription.deleted' webhook event is always 404. The reason is:

In the cancel_donation view, stripe_subscription_id is set to an empty string https://github.com/django/djangoproject.com/blob/e5a4bf20fb58c463441c15d552c022b03f716858/fundraising/views.py#L178

In the subscription_cancelled webhook handler method, the donation object is fetched using stripe_subscription_id which was set to an empty string earlier.

https://github.com/django/djangoproject.com/blob/e5a4bf20fb58c463441c15d552c022b03f716858/fundraising/views.py#L232-L233

So, the response will always be 404 and no cancellation email will be sent to the donor.

Guide contributeur

Direction de recherche
Examinez la vue cancel donation et le gestionnaire de webhook subscription cancelled. Le problème est que stripe subscription id est vidé avant que le webhook ne soit traité, ce qui provoque une erreur 404. Une solution pourrait consister à réorganiser les opérations ou à traiter le webhook avant d'effacer l'ID.
Stack technique
python
Domaine
backend
Type d'issue
Bug
DifficultéDifficulté estimée pour un nouveau contributeur, de 1 pour un très petit changement à 5 pour un travail expert.
2
Temps estiméFourchette de temps approximative pour investiguer, implémenter, tester et préparer une pull request.
1-3 heures
Statut d'activitéDisponibilité apparente de l'issue : fraîche, active, ancienne, bloquée ou en attente d'un mainteneur.
Active
ClartéClarté avec laquelle l'issue explique le changement attendu, les critères d'acceptation et la prochaine étape.
Claire
Prérequis
GitPython
Accessibilité débutantScore de 1 à 100 estimant l'accessibilité de cette issue pour un premier contributeur.
70