Project

General

Profile

Development #33869

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

Added by Emmanuel Cazenave 2 months ago. Updated 2 months ago.

Status:
Solution déployé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

0001-iparapheur-wcs-style-parameters-for-file-33869.patch View (9.22 KB) Emmanuel Cazenave, 19 Jun 2019 12:54 PM

0002-remove-python-magic-dependency-33869.patch View (990 Bytes) Emmanuel Cazenave, 19 Jun 2019 12:54 PM

0003-remove-outdated-README-33869.patch View (852 Bytes) Emmanuel Cazenave, 19 Jun 2019 12:54 PM

Associated revisions

Revision ecf095ce (diff)
Added by Emmanuel Cazenave 2 months ago

iparapheur: wcs style parameters for file (#33869)

Revision e44604fb (diff)
Added by Emmanuel Cazenave 2 months ago

remove python-magic dependency (#33869)

Revision 72e544e9 (diff)
Added by Emmanuel Cazenave 2 months ago

remove outdated README (#33869)

History

#1 Updated by Emmanuel Cazenave 2 months ago

#2 Updated by Serghei Mihai 2 months 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 2 months 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 2 months 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 2 months 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 2 months 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 months 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 months 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.

#9 Updated by Emmanuel Cazenave 2 months ago

Voilà pour le nettoyage de printemps sur ce endpoint uniquement, avec une utilisation de json-schema et à la poubelle python-magic.

#10 Updated by Frédéric Péters 2 months ago

  • Status changed from Solution proposée to Solution validée

Ok.

#11 Updated by Emmanuel Cazenave 2 months ago

  • Status changed from Solution validée to Résolu (à déployer)
commit 72e544e962cbb664c7c42357de5ede110626ffb5
Author: Emmanuel Cazenave <ecazenave@entrouvert.com>
Date:   Wed Jun 19 12:50:25 2019 +0200

    remove outdated README (#33869)

commit e44604fb1de97bfea0f48769c847675b5d2f7066
Author: Emmanuel Cazenave <ecazenave@entrouvert.com>
Date:   Wed Jun 19 12:49:39 2019 +0200

    remove python-magic dependency (#33869)

commit ecf095ce15574d05369a72902e09099a8d9f77a4
Author: Emmanuel Cazenave <ecazenave@entrouvert.com>
Date:   Wed Jun 12 14:20:16 2019 +0200

    iparapheur: wcs style parameters for file (#33869)

#12 Updated by Frédéric Péters 2 months ago

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

Also available in: Atom PDF