Produits Entr'ouvert » Passerelle

je suis d'avis de virer le get_magic_mime qui est une partie obscure. Et ainsi virer la dependance de python-magic.

J'étais pas partie pour faire l'adaptation demandée, mais en fait ça me semble bon à garder.

Par exemple pour faire tests du connecteurs hors wcs avec des types de fichiers différents sans se fatiguer à passer le mime type à la main.

Pour plus de lisibilité, je pense qu'on devrait quand même écrire un peu plus explicitement :

  if isinstance(data.get('file'), dict):  # w.c.s. style file
      content = base64.b64decode(data['file'].get('content'))
      content_type = data['file'].get('content_type')
  else:
      ...comme avant 

Aussi dans le test y'a un « 'content-type':'application/pdf' » qui marche pas, parce que c'est content_type qui est attendu. (bon, apparement ça a toujours été ainsi, tests foireux)

Sinon Serghei me suggère de balourder l'ancien format parce que c'était utilisé uniquement à Alf..., je suis plutôt d'accord, non ?

Emmanuel Cazenave a écrit :

Sinon Serghei me suggère de balourder l'ancien format parce que c'était utilisé uniquement à Alf..., je suis plutôt d'accord, non ?

Sur notre SaaS de prod on a effectivement aucune instance de ce connecteur... J'imagine que personne ailleurs n'a joué. Pour moi, ok pour un nettoyage.

Thomas Noël a écrit :

Emmanuel Cazenave a écrit :

Sinon Serghei me suggère de balourder l'ancien format parce que c'était utilisé uniquement à Alf..., je suis plutôt d'accord, non ?

Sur notre SaaS de prod on a effectivement aucune instance de ce connecteur... J'imagine que personne ailleurs n'a joué. Pour moi, ok pour un nettoyage.

Si on peut en profiter pour faire le tour des trucs qu'on voudrait changer tout de suite dans ce connecteur c'est pas plus mal (peut-être il y en a d'autre, je ne sais pas), ça évitera de traîner de mauvais choix trop longtemps.