Development #66453
développer le connecteur SIV
0%
Description
Sur la base de l'API de AAA-Data.
Fichiers
Révisions associées
Historique
Mis à jour par Serghei Mihai il y a presque 2 ans
Mis à jour par Serghei Mihai il y a presque 2 ans
- Fichier 0001-sivin-add-initial-app-66453.patch 0001-sivin-add-initial-app-66453.patch ajouté
- Statut changé de Nouveau à Solution proposée
- Patch proposed changé de Non à Oui
Mis à jour par Nicolas Roche il y a presque 2 ans
- Statut changé de Solution proposée à En cours
- Un appel pour rien est fait à call dans
consulterfinitionparimmat
. - En testant je vois l'API qui tombe en timeout (bien qu'un token me soit retourné) et donc je baserais le"
check status
" sur un appel à un endpoint plutôt que juste sur l'acquisition du token.
(Là l'API est HS ; je me demande si c'est pas moi qui l'ai cassée en ayant lancé une recherche sur le siren null :
https://passerelle.dev.publik.love/sivin/test/consulterflotteparsiren?siren=000000000 ) - Dommage de ne pas avoir les messages d'erreurs au format json dans l'APIError.
- Le test
test_no_api_key
renseigne deux tokens (alors qu'en fait aucun n'est retourné) : je retirerais le token non passant (TOKEN_401). - Les tests ne passent pas dans
get_infos_by_immat
.
Mis à jour par Nicolas Roche il y a presque 2 ans
(L'api est encore HS, mais cette fois-ci j'ai le timeout dès l'obtention du token.)
Mis à jour par Serghei Mihai il y a presque 2 ans
J'ai fait un mail à AAA-DATA et l'instance de recette est de nouveau opérationnelle.
Mis à jour par Serghei Mihai il y a presque 2 ans
- Fichier 0001-sivin-add-initial-app-66453.patch 0001-sivin-add-initial-app-66453.patch ajouté
- Statut changé de En cours à Solution proposée
Nicolas Roche a écrit :
- Un appel pour rien est fait à call dans
consulterfinitionparimmat
.
Corrigé.
- Dommage de ne pas avoir les messages d'erreurs au format json dans l'APIError.
Contrairement à la doc l'API ne retourne pas toujours du JSON. Exemple dans le test test_get_with_expired_token
.
- Le test
test_no_api_key
renseigne deux tokens (alors qu'en fait aucun n'est retourné) : je retirerais le token non passant (TOKEN_401).
Bien vu.
- Les tests ne passent pas dans
get_infos_by_immat
.
Rajouté.
Mis à jour par Nicolas Roche il y a presque 2 ans
- Statut changé de Solution proposée à Solution validée
Ok.
Juste que pour le endpoint "consulter flotte par siren", je préciserais qu'on reçoit un VIN :
<<< Get vehicles list of a SIREN --- Get VIN vehicule list of a SIREN >>>Sinon je vois des limitations, mais le connecteur n'y pourra rien :
- https://passerelle.dev.publik.love/sivin/test/consulterflotteparsiren?siren=000648006
on reçoit un VIN qui est inexploitable via l'API :WMA 3M166271
- Le endpoint "consulter finition scores par immat" répond "Finition non trouve" pour tous les plaques données en exemple dans la documentation.
- pas d'exemple qui utilise le codesra, mais rien qui ne l'explicite non plus dans la doc.
Mis à jour par Serghei Mihai il y a presque 2 ans
- Statut changé de Solution validée à Résolu (à déployer)
Nicolas Roche a écrit :
Ok.
Juste que pour le endpoint "consulter flotte par siren", je préciserais qu'on reçoit un VIN :
[...]
Ok, corrigé.
- Le endpoint "consulter finition scores par immat" répond "Finition non trouve" pour tous les plaques données en exemple dans la documentation.
Je l'ai constaté aussi. L'instance de recette est instable donc je mets ça sur le dos d'un jeu de données unstable.
- pas d'exemple qui utilise le codesra, mais rien qui ne l'explicite non plus dans la doc.
J'ai testé: avec ou sans code SRA l'API retourne la même chose. Tant pis.
Poussé:
commit 9543d149f0c083746e7128a961f70c7261524878 (HEAD -> main, origin/main, origin/HEAD) Author: Serghei Mihai <smihai@entrouvert.com> Date: Thu Jun 30 10:31:40 2022 +0200 translation update commit 30288c0c3278d08bf24574bfac0c777cb3c443ef Author: Serghei Mihai <smihai@entrouvert.com> Date: Mon Jun 20 17:32:16 2022 +0200 add connector for SIVIN (#66453)
Mis à jour par Transition automatique il y a presque 2 ans
- Statut changé de Résolu (à déployer) à Solution déployée
add connector for SIVIN (#66453)