JSON:API: 404/405 router errors are empty, and error objects have a non-spec `type` member
Les mainteneurs répondent en général sous 1 jour
Personne n'a encore pris cette issue.
Évaluation
- Difficulté
- 3/5
- Temps estimé
- 1-2 jours
- Accessibilité débutants
- 50/100
Piste de recherche
Start with JsonApi\Serializer\ErrorNormalizer in src/JsonApi/Serializer and reproduce with the Novel resource from the issue. Compare the bodies of GET /nope, DELETE /novels/1 and GET /novels/999 to find where the non-operation errors lose their status, title and detail. Done when the 404 and 405 errors carry the same members as the operation errors, and no error object has a type member while links.type remains.
Rédigé par le modèle d'indexation à partir du texte de l'issue.
Description
API Platform version(s) affected
4.4.4
Description
The JSON:API error documents have two problems that remain after #8366 (which makes status a string on main).
-
Errors that do not come from an operation are empty. An unknown route (404) or a method that is not allowed (405) answers with an error object that has only a generated
id. It has nostatus,titleordetail:{"errors":[{"id":"/.well-known/genid/1d2136a82a8b3a05d6a2"}]} -
Error objects have a
typemember. The spec lists the members of an error object (id,links,status,code,title,detail,source,meta), and "Unless otherwise noted, objects defined by this specification or any applied extensions MUST NOT contain any additional members."JsonApi\Serializer\ErrorNormalizercopies the problemtypetolinks.type, which is correct, but it also keeps it as atypemember:{"errors":[{"id":"/errors/404","title":"An error occurred","detail":"Not Found","status":404,"type":"/errors/404","links":{"type":"/errors/404"},"code":"404"}]}
#3042 is about the older problem+json format. These two problems are in the current errors format.
How to reproduce
#[ApiResource(operations: [
new Get(provider: [self::class, 'provide']),
new Post(processor: [self::class, 'process']),
])]
final class Novel
{
public int $id = 1;
public string $title = 'Dune';
public static function provide(): ?self { return null; }
public static function process(self $data, Operation $operation): self { return $data; }
}
| Request | Status | Body |
|---|---|---|
GET /nope |
404 | {"errors":[{"id":"/.well-known/genid/…"}]} |
DELETE /novels/1 |
405 | {"errors":[{"id":"/.well-known/genid/…"}]} |
GET /novels/999 |
404 | {"errors":[{"id":"/errors/404","title":"An error occurred","detail":"Not Found","status":404,"type":"/errors/404","links":{"type":"/errors/404"},"code":"404"}]} |
POST /novels with {"data":{"type":"Novel","attributes":{"title":5}}} |
400 | {"errors":[{"id":"/errors/400","title":"An error occurred","detail":"The type of the \"title\" attribute must be \"string\", \"integer\" given.","status":400,"type":"/errors/400","links":{"type":"/errors/400"},"code":"400"}]} |
Expected:
- The 404 and 405 errors have the same members as the other errors:
status,title,detail,codeandlinks.type. - Error objects have no
typemember. The value stays inlinks.type.
Additional Context
Checked on api-platform/core v4.4.4, Symfony 7.4, PHP 8.5, in an empty Symfony app. The related code in src/JsonApi/Serializer is the same on main, except for the error status cast (#8366).
The app has only FrameworkBundle, TwigBundle and ApiPlatformBundle, with this config:
$c->extension('api_platform', [
'mapping' => ['paths' => ['%kernel.project_dir%/src/ApiResource']],
'formats' => ['jsonapi' => ['application/vnd.api+json']],
'error_formats' => ['jsonapi' => ['application/vnd.api+json'], 'jsonproblem' => ['application/problem+json']],
'doctrine' => ['enabled' => false],
]);
All requests send Accept: application/vnd.api+json and, when they have a body, Content-Type: application/vnd.api+json. The responses are from the prod environment.
- Langage dominant
- PHP
- Étoiles
- 2.6k
- Forks
- 989
- Merge moyen
- 2 j 2 h
- PR mergées (30 j)
- 101
Préparer son environnement
- Aucun Dockerfile ni fichier Docker Compose
- Propose un modèle de pull request
- Lire le guide de contribution
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 api-platform/core
-
JSON:API: resource schemas (`*.jsonapi`) have `properties` but no `type: object`Peut-être pris Une pull request liée à cette issue est ouverte ou déjà fusionnée. Ouverte
Difficulté 2/5 Moins d'une heure Accessibilité débutants 80/100
api-platform/core#8701 ·
Les mainteneurs répondent en général sous 1 jour
-
JSON:API: `JsonApiCollectionBaseSchemaNoPagination` declares `data` as `type: array` without `items`Ouverte
Difficulté 2/5 1-3 heures Accessibilité débutants 82/100
api-platform/core#8699 ·
Les mainteneurs répondent en général sous 1 jour
-
api:openapi:export --spec-version=3.0.0 outputs `webhooks`, which is invalid in OpenAPI 3.0Peut-être pris Une pull request liée à cette issue est ouverte ou déjà fusionnée. Ouverte
Difficulté 2/5 1-3 heures Accessibilité débutants 72/100
api-platform/core#8690 ·
Les mainteneurs répondent en général sous 1 jour
-
Difficulté 1/5 Moins d'une heure Accessibilité débutants 85/100
api-platform/core#8680 ·
Les mainteneurs répondent en général sous 1 jour
-
JSON:API: request documents with no `data`, no `type` or a wrong `type` are accepted with 201Ouverte
Difficulté 3/5 1-2 jours Accessibilité débutants 62/100
api-platform/core#8703 ·
Les mainteneurs répondent en général sous 1 jour
Toutes les issues de api-platform/core
Issues similaires
-
Difficulté 2/5 1-3 heures Accessibilité débutants 78/100
-
Featured image not passed to checks in the Tasks column or publish blocksPeut-être pris @roborourke l’a pris aujourd’hui. Ouverte
Difficulté 2/5 1-3 heures Accessibilité débutants 78/100
humanmade/publication-checklist#100 ·
Les mainteneurs répondent en général sous 1 jour
-
bug customer-reported
Difficulté 2/5 1-3 heures Accessibilité débutants 62/100
MagnaCapax/PMSS#1072 ·
Les mainteneurs répondent en général sous 5 jours
-
Difficulté 2/5 1-3 heures Accessibilité débutants 82/100
verbb/formie#3013 · 1 commentaire ·
Les mainteneurs répondent en général sous 1 jour
-
Documentation Platform: Plugin
Difficulté 2/5 1-3 heures Accessibilité débutants 83/100
Les mainteneurs répondent en général sous 1 jour