Development #66453
développer le connecteur SIV
0%
Description
Sur la base de l'API de AAA-Data.
Files
Associated revisions
History
Updated by Serghei Mihai 12 months ago
Updated by Serghei Mihai 12 months ago
- File 0001-sivin-add-initial-app-66453.patch 0001-sivin-add-initial-app-66453.patch added
- Status changed from Nouveau to Solution proposée
- Patch proposed changed from No to Yes
Updated by Nicolas Roche 12 months ago
- Status changed from Solution proposée to 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
.
Updated by Nicolas Roche 12 months ago
(L'api est encore HS, mais cette fois-ci j'ai le timeout dès l'obtention du token.)
Updated by Serghei Mihai 12 months ago
J'ai fait un mail à AAA-DATA et l'instance de recette est de nouveau opérationnelle.
Updated by Serghei Mihai 12 months ago
- File 0001-sivin-add-initial-app-66453.patch 0001-sivin-add-initial-app-66453.patch added
- Status changed from En cours to 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é.
Updated by Nicolas Roche 11 months ago
- Status changed from Solution proposée to 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.
Updated by Serghei Mihai 11 months ago
- Status changed from Solution validée to 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)
Updated by Transition automatique 11 months ago
- Status changed from Résolu (à déployer) to Solution déployée
add connector for SIVIN (#66453)