Project

General

Profile

Development #33869

iparapheur : accepter le format renvoyé par le serializer json de wcs

Added by Emmanuel Cazenave 4 days ago. Updated 2 days ago.

Status:
Solution proposée
Priority:
Normal
Target version:
-
Start date:
12 Jun 2019
Due date:
% Done:

0%

Patch proposed:
Yes
Planning:
No

Description

Actuellement sur le endpoint create_file il faut passer le contenu du fichier en base64 dans le paramètre 'data', ce qui oblige à faire des contorsions dans l'action d'appel webservice, genre :

data : form_var_doc_raw.get_content().encode('base64')

Le JSONEncoder de wcs fait ça tout seul.

0001-iparapheur-accept-wcs-style-parameters-33869.patch View (5.05 KB) Emmanuel Cazenave, 12 Jun 2019 02:21 PM

History

#1 Updated by Emmanuel Cazenave 3 days ago

#2 Updated by Serghei Mihai 3 days ago

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

#3 Updated by Emmanuel Cazenave 3 days ago

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.

#4 Updated by Thomas Noël 3 days ago

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 

#5 Updated by Thomas Noël 3 days ago

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)

#6 Updated by Emmanuel Cazenave 3 days ago

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

#7 Updated by Thomas Noël 2 days ago

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.

#8 Updated by Benjamin Dauvergne 2 days ago

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.

Also available in: Atom PDF