entrouvert
/
passerelle
15
0
Fork 0
Code Issues Pull Requests 15 Releases Activity

atal_rest: add multiple attachment endpoint (#79140) #302

Merged
ecazenave merged 1 commits from wip/79140-atal-rest-attachments into main 2023-06-28 15:10:28 +02:00
Conversation 2 Commits 1 Files Changed 2 +127 -14
ecazenave commented 2023-06-28 11:43:02 +02:00
Owner
Copy Link
No description provided.
ecazenave added 1 commit 2023-06-28 11:43:03 +02:00
gitea/passerelle/pipeline/head This commit looks good Details
462aea72cf atal_rest: add multiple attachment endpoint (#79140)
ecazenave changed title from WIP: atal_rest: add multiple attachment endpoint (#79140) to atal_rest: add multiple attachment endpoint (#79140) 2023-06-28 12:00:48 +02:00
ecazenave commented 2023-06-28 12:02:19 +02:00
Author
Owner
Copy Link

Désolé pour le diff un peu foutraque.

Au passage ajout de 'input_exemple' et un petit ajustement nécessaire dans las méthode _call.

Et flemme de faire de commits séparés pour ce connecteur tout frais sorti hier.

Désolé pour le diff un peu foutraque. Au passage ajout de 'input_exemple' et un petit ajustement nécessaire dans las méthode _call. Et flemme de faire de commits séparés pour ce connecteur tout frais sorti hier.
pmarillonnet approved these changes 2023-06-28 15:03:00 +02:00
pmarillonnet left a comment
Owner
Copy Link

Ok, vu avec Manu de vive voix, j’avais un bout de code qui me laissait perplexe

        for file_ in post_data.get('files', []):
            # […]
            files.append(
                (
                    'Files',
                    (
                        file_.get('filename', ''),
                        io.BytesIO(content).read(),
                        file_.get('content_type', ''),
                    ),
                )

mais c’est apparemment une façon de requests de gérer l’envoi de fichier en multipart, en accord avec ce qu’attend Atal à l’autre bout du tuyau.

Bref, ack.

Ok, vu avec Manu de vive voix, j’avais un bout de code qui me laissait perplexe ```python for file_ in post_data.get('files', []): # […] files.append( ( 'Files', ( file_.get('filename', ''), io.BytesIO(content).read(), file_.get('content_type', ''), ), ) ``` mais c’est apparemment une façon de `requests` de gérer l’envoi de fichier en multipart, en accord avec ce qu’attend Atal à l’autre bout du tuyau. Bref, ack.
ecazenave commented 2023-06-28 15:10:23 +02:00
Author
Owner
Copy Link

Pour mémoire : https://requests.readthedocs.io/en/latest/user/advanced/#post-multiple-multipart-encoded-files.

Pour mémoire : https://requests.readthedocs.io/en/latest/user/advanced/#post-multiple-multipart-encoded-files.
ecazenave merged commit a16dc0c83a into main 2023-06-28 15:10:28 +02:00
ecazenave deleted branch wip/79140-atal-rest-attachments 2023-06-28 15:10:29 +02:00
Sign in to join this conversation.
Reviewers
No reviewers
pmarillonnet
Labels
Clear labels
No items
No Label
Milestone
Clear milestone
No items
No Milestone
Assignees
Clear assignees
No Assignees
2 Participants
Notifications
Due Date
The due date is invalid or out of range. Please use the format 'yyyy-mm-dd'.

No due date set.

Dependencies

No dependencies set.

Reference: entrouvert/passerelle#302
No description provided.
Delete Branch "wip/79140-atal-rest-attachments"

Deleting a branch is permanent. Although the deleted branch may continue to exist for a short time before it actually gets removed, it CANNOT be undone in most cases. Continue?