Projet

Général

Profil

Development #66453

développer le connecteur SIV

Ajouté par Serghei Mihai il y a presque 2 ans. Mis à jour il y a presque 2 ans.

Statut:
Fermé
Priorité:
Normal
Assigné à:
Version cible:
-
Début:
21 juin 2022
Echéance:
% réalisé:

0%

Temps estimé:
Patch proposed:
Oui
Planning:
Non

Description

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


Fichiers

Révisions associées

Révision 30288c0c (diff)
Ajouté par Serghei Mihai il y a presque 2 ans

add connector for SIVIN (#66453)

Historique

#3

Mis à jour par Serghei Mihai il y a presque 2 ans

#4

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 .
#5

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

#6

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.

#7

Mis à jour par Serghei Mihai il y a presque 2 ans

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

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 :
#9

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)

#10

Mis à jour par Transition automatique il y a presque 2 ans

  • Statut changé de Résolu (à déployer) à Solution déployée
#11

Mis à jour par Transition automatique il y a plus d'un an

Automatic expiration

Formats disponibles : Atom PDF