hub.verify not in spec anymore, still required by hub on subscribe
Personne n'a encore pris cette issue.
Évaluation
- Difficulté
- 3/5
- Temps estimé
- 1-2 jours
- Accessibilité débutants
- 35/100
Piste de recherche
Commencez dans class-pushpress.php à la ligne 49 et comparez la gestion des abonnements avec la spécification actuelle de WebSub 0.4 liée dans l’issue. Suivez la manière dont le plugin valide les paramètres d’abonnement et vérifiez qu’un client à jour n’est plus rejeté avec HTTP 400 parce que hub.verify est absent.
Rédigé par le modèle d'indexation à partir du texte de l'issue.
Description
I'm using flask_websub (https://readthedocs.org/projects/flask-websub/) as a subscriber for a wordpress multisite on which I've installed the pushpress plugin. Comparing websub drafts 0.3 (https://pubsubhubbub.github.io/PubSubHubbub/pubsubhubbub-core-0.3.html) and current 0.4 (https://www.w3.org/TR/websub/) it seems that parameter hub.verify is not required anymore at subscribe. The hub included in pushpress still looks for it (line 49 of https://github.com/Automattic/pushpress/blob/master/class-pushpress.php) returning 400 to any (up to date) client.
I'll probably hack my way around this client side to conform to current hub status, but the best option would still be updating the plugin's behavior.
I'm not very php savvy, but if someone's willing to hold my hand through a review and/or tests I could try a pull request.
- Langage dominant
- PHP
- Étoiles
- 15
- Forks
- 10
- Métriques de merge des PR
- Aucune PR mergée en 30 j
Préparer son environnement
Ce projet ne fournit ni conteneur de développement, ni Dockerfile, ni guide de contribution : l'installation est à votre charge. Commencez par son README, et consultez notre guide de la première contribution pour les étapes générales.
Par où commencer
- Lisez l'issue en entier, puis le guide de contribution du projet.
- Signalez en commentaire que vous la prenez — cela évite que deux personnes fassent le même travail.
- Forkez le dépôt et travaillez sur une branche.
- Ouvrez une pull request qui référence le numéro de l'issue.
Autres issues de Automattic/pushpress
-
Difficulté 1/5 1-3 heures Accessibilité débutants 85/100
Automattic/pushpress#18 ·
-
Difficulté 2/5 1-3 heures Accessibilité débutants 83/100
Automattic/pushpress#17 ·
-
GUID normalisation bug, leading to duplicatesPeut-être pris @Alkarex l’a pris il y a 421 jours. Ouverte
Difficulté 3/5 1-2 jours Accessibilité débutants 42/100
Automattic/pushpress#15 · 1 commentaire · 1 réaction ·
-
enhancement help wanted
Difficulté 5/5 Plus d'une semaine Accessibilité débutants 30/100
Automattic/pushpress#14 ·
Toutes les issues de Automattic/pushpress
Issues similaires
-
Difficulté 2/5 1-3 heures Accessibilité débutants 84/100
Les mainteneurs répondent en général sous 3 jours
-
sync-en
Difficulté 2/5 1-3 heures Accessibilité débutants 74/100
Les mainteneurs répondent en général sous 1 jour
-
[Sync EN] Formatting improvements to "Large Objects (LOBs)" docs (#6056)Peut-être pris Une pull request liée à cette issue est ouverte ou déjà fusionnée. Ouvertesync-en
Difficulté 1/5 Moins d'une heure Accessibilité débutants 74/100
Les mainteneurs répondent en général sous 1 jour
-
Перевод устарел
Difficulté 2/5 1-3 heures Accessibilité débutants 73/100
Les mainteneurs répondent en général sous 1 jour
-
repo:cms
Difficulté 2/5 1-3 heures Accessibilité débutants 78/100
craftcms/cms#20613 · 1 commentaire ·
Les mainteneurs répondent en général sous 1 jour