[Schema Inaccuracy] `X-GitHub-Hook-Installation-Target-Type` and `X-GitHub-Hook-Installation-Target-Id` are insufficiently documented (and incorrect)
Los mantenedores suelen responder en 1 día
Nadie ha tomado este issue todavía.
Evaluación
- Dificultad
- 3/5
- Tiempo estimado
- 1-2 días
- Aptitud para principiantes
- 68/100
- Tipo de issue
- Error
- Claridad
- Bastante claro
- Estado de actividad
- Activo
- Stack tecnológico
- openapi
- Área
- api
Línea de trabajo
Comienza con las dos entradas del esquema de encabezados de webhook incluidas en este issue y compáralas con la documentación vinculada de delivery-headers y los informes anteriores. Verifica los valores válidos de Target-Type y la correspondencia de Target-Id para cada una, y luego actualiza los tipos y las descripciones del esquema para que los valores y los identificadores documentados sean precisos.
Escrito por el modelo de indexación a partir del texto del issue.
Descripción
Schema Inaccuracy
The schema for these two webhook headers only specifies:
{
"name": "X-Github-Hook-Installation-Target-Id",
"in": "header",
"example": 123123,
"schema": {
"type": "string"
}
},
{
"name": "X-Github-Hook-Installation-Target-Type",
"in": "header",
"example": "repository",
"schema": {
"type": "string"
}
},
the webhooks documentation is not significantly more helpful:
X-GitHub-Hook-Installation-Target-Type: The type of resource where the webhook was created.
X-GitHub-Hook-Installation-Target-ID: The unique identifier of the resource where the webhook was created.
Issue
While some hooks can be distinguished by probing the payload for optional entries like installation or organization entries, not only is this pretty ugly some events don't have such entries because e.g. they're not attached to installations in the case of github apps hooks (ping, github_app_authorization.revoked).
Currently every developer who needs this information has to reverse-engineer the values and hope they didn't miss special cases.
Expected
- enumerate the valid values for
Target-Type(as far as I can tell there'srepositoryfor repository hooks andintegrationfor app hooks, I have never tested org hooks so I've no idea what that uses - specify what
Target-Idmaps to for each type, as far as I can tell forrepositoryit's the repository id and forintegrationit's the app id - also the
Target-Idis currently typed as astring, but all of them are numbers (/ integers)
Previous reports
- Lenguaje dominante
- Sin datos de lenguaje
- Estrellas
- 1.6k
- Forks
- 345
- Merge medio
- 6 h
- PR fusionados (30 d)
- 82
Preparar el entorno
- Sin Dockerfile ni archivo de Docker Compose
- Tiene una plantilla de pull request
- Leer la guía de contribución
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 github/rest-api-description
-
Dificultad 1/5 Menos de una hora Aptitud para principiantes 88/100
github/rest-api-description#7266 · 1 comentario ·
Los mantenedores suelen responder en 1 día
-
feature
Dificultad 2/5 1-3 horas Aptitud para principiantes 82/100
github/rest-api-description#7246 ·
Los mantenedores suelen responder en 1 día
-
feature
Dificultad 2/5 1-3 horas Aptitud para principiantes 85/100
github/rest-api-description#7220 ·
Los mantenedores suelen responder en 1 día
-
feature
Dificultad 2/5 1-3 horas Aptitud para principiantes 82/100
github/rest-api-description#7201 ·
Los mantenedores suelen responder en 1 día
-
feature
Dificultad 2/5 1-3 horas Aptitud para principiantes 72/100
github/rest-api-description#7163 ·
Los mantenedores suelen responder en 1 día
Todos los issues de github/rest-api-description
Issues similares
-
company delete fails with 500 on any company that has activity (cost events, inbox dismissals)Abierto
Dificultad 2/5 1-3 horas Aptitud para principiantes 84/100
paperclipai/paperclip#14982 ·
Los mantenedores suelen responder en 1 día
-
cbor
Dificultad 2/5 1-3 horas Aptitud para principiantes 84/100
FasterXML/jackson-dataformats-binary#844 ·
Los mantenedores suelen responder en 1 día
-
HasBacktrace Priority-Critical
Dificultad 2/5 1-3 horas Aptitud para principiantes 78/100
azerothcore/azerothcore-wotlk#27921 ·
Los mantenedores suelen responder en 1 día
-
Dificultad 2/5 1-3 horas Aptitud para principiantes 84/100
lawndoc/stack-back#123 ·
-
type/bug
Dificultad 2/5 1-3 horas Aptitud para principiantes 78/100
Los mantenedores suelen responder en 1 día