hub.verify not in spec anymore, still required by hub on subscribe
Nadie ha tomado este issue todavía.
Evaluación
- Dificultad
- 3/5
- Tiempo estimado
- 1-2 días
- Aptitud para principiantes
- 35/100
Línea de trabajo
Comienza en class-pushpress.php en la línea 49 y compara el manejo de las suscripciones con la especificación actual de WebSub 0.4 enlazada en la issue. Traza cómo el plugin valida los parámetros de suscripción y verifica que un cliente actualizado ya no sea rechazado con HTTP 400 porque falta hub.verify.
Escrito por el modelo de indexación a partir del texto del issue.
Descripción
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.
- Lenguaje dominante
- PHP
- Estrellas
- 15
- Forks
- 10
- Métricas de merge de PR
- Sin PR fusionados en 30 d
Preparar el entorno
Este proyecto no incluye contenedor de desarrollo, Dockerfile ni guía de contribución, así que la configuración corre por tu cuenta: empieza por su README y consulta nuestra guía para la primera contribución para los pasos generales.
Primeros pasos
- Lee el issue completo y luego la guía de contribución del proyecto.
- Comenta en el issue que vas a ocuparte — evita que dos personas hagan lo mismo.
- Haz un fork del repositorio y trabaja en una rama.
- Abre un pull request que haga referencia al número del issue.
Más de Automattic/pushpress
-
Dificultad 1/5 1-3 horas Aptitud para principiantes 85/100
Automattic/pushpress#18 ·
-
Dificultad 2/5 1-3 horas Aptitud para principiantes 83/100
Automattic/pushpress#17 ·
-
GUID normalisation bug, leading to duplicatesPosiblemente ocupada @Alkarex la tomó hace 420 días. Abierto
Dificultad 3/5 1-2 días Aptitud para principiantes 42/100
Automattic/pushpress#15 · 1 comentario · 1 reacción ·
-
enhancement help wanted
Dificultad 5/5 Más de una semana Aptitud para principiantes 30/100
Automattic/pushpress#14 ·
Todos los issues de Automattic/pushpress
Issues similares
-
Dificultad 2/5 1-3 horas Aptitud para principiantes 84/100
Los mantenedores suelen responder en 3 días
-
sync-en
Dificultad 2/5 1-3 horas Aptitud para principiantes 74/100
Los mantenedores suelen responder en 1 día
-
[Sync EN] Formatting improvements to "Large Objects (LOBs)" docs (#6056)Posiblemente ocupada Un pull request vinculado a esta issue está abierto o ya se fusionó. Abiertosync-en
Dificultad 1/5 Menos de una hora Aptitud para principiantes 74/100
Los mantenedores suelen responder en 1 día
-
Перевод устарел
Dificultad 2/5 1-3 horas Aptitud para principiantes 73/100
Los mantenedores suelen responder en 1 día
-
repo:cms
Dificultad 2/5 1-3 horas Aptitud para principiantes 78/100
craftcms/cms#20613 · 1 comentario ·
Los mantenedores suelen responder en 1 día