Creating a destination with an existing ID returns 400
Los mantenedores suelen responder en 2 días
Nadie ha tomado este issue todavía.
Evaluación
- Dificultad
- 2/5
- Tiempo estimado
- 1-3 horas
- Aptitud para principiantes
- 68/100
Línea de trabajo
Comienza en el handler de POST /api/v1/tenants/:tenant_id/destinations y encuentra dónde escribe 400 para un destination id existente y para el tenant destination limit; compara con el handler de POST /publish que ya devuelve 409 para un event id en conflicto. Actualiza ambos status codes, ajusta los tests del handler para que asserten 409 y añade una release note que señale el breaking change para los clients que hacen match en 400.
Escrito por el modelo de indexación a partir del texto del issue.
Descripción
POST /api/v1/tenants/:tenant_id/destinations with an id that a live destination already has returns:
{"status":400,"message":"destination already exists"}
409 Conflict is the usual status for this: the request is well-formed, and it conflicts with the current state. The API already uses 409 on POST /publish for a conflicting event id.
The same handler returns 400 when the tenant has reached its destination limit, which is also a state conflict more than a bad request.
Changing either status breaks clients that match on 400, so it needs a release note. For reference, PUT /tenants/:tenant_id has no such case: it is an upsert and returns 200 for an existing tenant, 201 for a new one.
- Lenguaje dominante
- Go
- Estrellas
- 1.1k
- Forks
- 70
- Merge medio
- 2 d 20 h
- PR fusionados (30 d)
- 44
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 hookdeck/outpost
-
Dificultad 2/5 1-3 horas Aptitud para principiantes 76/100
Los mantenedores suelen responder en 2 días
-
Dificultad 3/5 1-2 días Aptitud para principiantes 67/100
Los mantenedores suelen responder en 2 días
-
enhancement
Dificultad 4/5 3-5 días Aptitud para principiantes 48/100
hookdeck/outpost#1117 · 1 comentario · 1 reacción ·
Los mantenedores suelen responder en 2 días
-
Dificultad 4/5 3-5 días Aptitud para principiantes 35/100
Los mantenedores suelen responder en 2 días
-
Dificultad 4/5 3-5 días Aptitud para principiantes 45/100
Los mantenedores suelen responder en 2 días
Todos los issues de hookdeck/outpost
Issues similares
-
area: records fast-track-debt priority: normal
Dificultad 2/5 1-3 horas Aptitud para principiantes 76/100
Los mantenedores suelen responder en 1 día
-
processor/genainormalizer
Dificultad 2/5 1-3 horas Aptitud para principiantes 64/100
open-telemetry/opentelemetry-collector-contrib#52328 · 1 comentario ·
Los mantenedores suelen responder en 1 día
-
[tooling] Example acceptance runner reports ALL PASS for an unsupported program (tested patch)Abierto
Dificultad 2/5 1-3 horas Aptitud para principiantes 82/100
limitlessprojects/Go-Hybrid#1595 ·
Los mantenedores suelen responder en 1 día
-
provider: hyper
Dificultad 2/5 1-3 horas Aptitud para principiantes 78/100
charmbracelet/crush#4134 ·
Los mantenedores suelen responder en 1 día
-
bug
Dificultad 2/5 1-3 horas Aptitud para principiantes 66/100
AvengeMedia/DankMaterialShell#3912 ·