Project

General

Profile

Development #66453

développer le connecteur SIV

Added by Serghei Mihai 12 days ago. Updated 3 days ago.

Status:
Solution déployée
Priority:
Normal
Assignee:
Target version:
-
Start date:
21 Jun 2022
Due date:
% Done:

0%

Estimated time:
Patch proposed:
Yes
Planning:
No

Description

Sur la base de l'API de AAA-Data.


Files

Associated revisions

Revision 30288c0c (diff)
Added by Serghei Mihai 3 days ago

add connector for SIVIN (#66453)

History

#3

Updated by Serghei Mihai 9 days ago

#4

Updated by Nicolas Roche 6 days 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 .
#5

Updated by Nicolas Roche 5 days ago

(L'api est encore HS, mais cette fois-ci j'ai le timeout dès l'obtention du token.)

#6

Updated by Serghei Mihai 5 days ago

J'ai fait un mail à AAA-DATA et l'instance de recette est de nouveau opérationnelle.

#7

Updated by Serghei Mihai 5 days ago

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é.

#8

Updated by Nicolas Roche 4 days 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 :
#9

Updated by Serghei Mihai 3 days 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)

#10

Updated by Transition automatique 3 days ago

  • Status changed from Résolu (à déployer) to Solution déployée

Also available in: Atom PDF