openapi: fix openapi validation errors and json validation from requests - [merged]

[benoitdm-oslandia]

_Merges fix/invalidopenapi -> develop

• upgrade kin-openapi version
  • replace Swagger type by T type
• improve geojson validation by extracting openapi schema from https://geojson.org/schema/XXX.json urls (see makeGeojsonSchemaRefs)
• fix examples to respect described schema
• extract description as they need to be pointer

close #64

[benoitdm-oslandia]

added 1 commit

• 2fc12e67 - openapi: fix openapi validation errors and json validation from requests

Compare with previous version

[benoitdm-oslandia]

added 1 commit

• 04d38126 - openapi: fix openapi validation errors and json validation from requests

Compare with previous version

[benoitdm-oslandia]

added 2 commits

• ed04842d - 1 commit from branch develop
• 8b7e9d62 - openapi: fix openapi validation errors and json validation from requests

Compare with previous version

[benoitdm-oslandia]

added 3h of time spent

[benoitdm-oslandia]

In GitLab by @azarz on Oct 13, 2022, 14:38

approved this merge request

[benoitdm-oslandia]

In GitLab by @azarz on Oct 13, 2022, 14:44

Commented on internal/api/openapi.go line 431

Pourquoi il y a un "1" ici ?

En curl ça donne &transform=Centroid|Buffer|1, c'est bizarre non ?

J'imagine que c'est pour donner en entrée le paramètre au buffer, mais ça ne semble pas fonctionner (Invalid value for parameter transform: 1)

[benoitdm-oslandia]

je suis bien d'accord! C'est moche :smile:

Je suis parti du principe que la virgule d'origine servait à séparer les 2 cas d'usages. Après je ne me suis pas penché sur le fonctionnement des transforms pour savoir comment l'exemple devait être interprété.

[benoitdm-oslandia]

In GitLab by @azarz on Oct 13, 2022, 14:52

Commented on internal/api/openapi.go line 431

Je viens de me plonger dans le code ! A priori ce serai plutôt, dans le curl, &transform=Centroid|Buffer,1.

Vu que le style de la liste est pipeDelimited, l'example doit plutôt être

            Example: []string{"Centroid", "Buffer,1"},

[benoitdm-oslandia]

In GitLab by @azarz on Oct 13, 2022, 14:54

unapproved this merge request

[benoitdm-oslandia]

hum ... quel est le but de cette transform ?

Peut être faire un buffer de 1 à partir du centroid ? parce qu'une transform qui ne fait que centroid c'est bizarre ... donc je laisserais Centroid|Buffer ensemble, après le 1 je ne sais pas ... soit c un parametre de buffer soit c'est un truc par défaut qui veut dire que c'est une transform qui ne fait rien ...

Peut être qu'il faut laisser tout ensemble : Example: []string{"Centroid|Buffer,1"}, ?

[benoitdm-oslandia]

In GitLab by @azarz on Oct 13, 2022, 15:05

Commented on internal/api/openapi.go line 431

D'après le code de param.go, | sépare les noms de transform, et , les paramèrtres des fonctions. Transform qui ne fait que Centroid c'est imaginable pour des requêtes sur des polygones je trouve. Après le cas d'usage de l'exemple ne dois pas exister (fairee un buffer de 1 unité autour du centroid), mais je pense que c'est justement pour l'exemple.

Pour moi la présence du champ Style: "pipeDelimited" montre bien que c'est une liste dont le délimiteur est | dans le paramètre, donc je reste sur mon interprétation

[benoitdm-oslandia]

In GitLab by @azarz on Oct 13, 2022, 15:05

Commented on internal/api/openapi.go line 431

https://git.oslandia.net/Client-projects/geoplateforme-ign-pg-featureserv/-/blob/develop/internal/service/param.go#L360

[benoitdm-oslandia]

ok !

[benoitdm-oslandia]

resolved all threads

[benoitdm-oslandia]

changed this line in version 5 of the diff

[benoitdm-oslandia]

added 1 commit

• 4c89c7cd - openapi: fix openapi validation errors and json validation from requests

Compare with previous version

[benoitdm-oslandia]

In GitLab by @azarz on Oct 13, 2022, 15:18

approved this merge request

[benoitdm-oslandia]

added 3 commits

• 4c89c7cd...944fcc36 - 2 commits from branch develop
• 58464f01 - openapi: fix openapi validation errors and json validation from requests

Compare with previous version

[benoitdm-oslandia]

added 3 commits

• 58464f01...5f30aec0 - 2 commits from branch develop
• c1ddb739 - openapi: fix openapi validation errors and json validation from requests

Compare with previous version

[benoitdm-oslandia]

enabled an automatic merge when the pipeline for c1ddb739ad415c8c9798ed33c5e75baf54edfd67 succeeds