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

filr_rest: start connector (#73226) #34

Closed
ecazenave wants to merge 0 commits from wip/73226-filr-rest into main
pull from: wip/73226-filr-rest
merge into: entrouvert:main
Conversation 1 Commits - Files Changed -
ecazenave commented 2023-01-10 14:59:52 +01:00
Owner
Copy Link
No description provided.
ecazenave force-pushed wip/73226-filr-rest from a888787597 to c9628be6cd 2023-01-10 15:11:49 +01:00 Compare
Ghost reviewed 2023-01-10 16:03:03 +01:00
Ghost reviewed 2023-01-10 16:05:15 +01:00
passerelle/apps/filr_rest/models.py Outdated
@ -0,0 +35,4 @@
if params:
kwargs['params'] = params
try:
resp = getattr(self.requests, method)(urllib.parse.urljoin(self.base_url, path), **kwargs)
Ghost commented 2023-01-10 16:05:15 +01:00
First-time contributor
Copy Link

Tu peux aussi faire:

   resp = self.requests.request(method=method, url=urllib.parse.urljoin(self.base_url, path), **kwargs)
Tu peux aussi faire: ``` resp = self.requests.request(method=method, url=urllib.parse.urljoin(self.base_url, path), **kwargs) ```
ecazenave commented 2023-01-10 16:55:24 +01:00
Author
Owner
Copy Link

tenu compte

tenu compte
Ghost marked this conversation as resolved
ecazenave force-pushed wip/73226-filr-rest from c9628be6cd to d4683bcf33 2023-01-10 16:05:25 +01:00 Compare
Ghost reviewed 2023-01-10 16:14:50 +01:00
passerelle/apps/filr_rest/schemas.py Outdated
@ -0,0 +25,4 @@
},
'filename': {
'type': 'string',
'description': _('Filename (takes precendence over filename in "file" object)'),
Ghost commented 2023-01-10 16:14:50 +01:00
First-time contributor
Copy Link

s/precendence/precedence

s/precendence/precedence
ecazenave commented 2023-01-10 16:55:33 +01:00
Author
Owner
Copy Link

tenu compte

tenu compte
Ghost marked this conversation as resolved
ecazenave force-pushed wip/73226-filr-rest from d4683bcf33 to 5a6abae09d 2023-01-10 16:23:14 +01:00 Compare
ecazenave force-pushed wip/73226-filr-rest from 5a6abae09d to c66d57f6b3 2023-01-10 16:55:10 +01:00 Compare
ecazenave commented 2023-01-10 16:56:14 +01:00
Author
Owner
Copy Link

Je laisse en wip pour l'instant, je demande un accès à une plateforme de test pour tester en vrai, quand même.

Je laisse en wip pour l'instant, je demande un accès à une plateforme de test pour tester en vrai, quand même.
Ghost requested changes 2023-01-16 17:09:44 +01:00
Ghost left a comment
First-time contributor
Copy Link

il faut corriger le retour 204 (vide) pour delete-folder

il faut corriger le retour 204 (vide) pour delete-folder
passerelle/apps/filr_rest/models.py Outdated
@ -0,0 +66,4 @@
raise APIError(str(main_exc), data=err_data)
content_type = resp.headers.get('Content-Type')
if content_type and content_type.startswith('application/json'):
Ghost commented 2023-01-16 17:04:54 +01:00
First-time contributor
Copy Link

Il faut gérer ici le status 204 qui signifie une réponse OK mais vide, et pour laquelle le resp.json() va planter. C'est nécessaire pour delete-folder

Il faut gérer ici le status 204 qui signifie une réponse OK mais vide, et pour laquelle le resp.json() va planter. C'est nécessaire pour delete-folder
ecazenave commented 2023-01-17 14:28:51 +01:00
Author
Owner
Copy Link

Tenu compte (et adaptation du test d'abord pour reproduire).

Tenu compte (et adaptation du test d'abord pour reproduire).
Ghost marked this conversation as resolved
passerelle/apps/filr_rest/models.py Outdated
@ -0,0 +116,4 @@
name='share-folder',
perm='can_access',
description=_('Share a folder to external users'),
post={'request_body': {'schema': {'application/json': schemas.SHARE_FOLDER}}},
Ghost commented 2023-01-16 17:09:10 +01:00
First-time contributor
Copy Link

Le schéma donne un affichage cabalistique:

Share a folder to external users : /filr-rest/cd44/share-folder

    Corps de la requête
        application/json : object, désaplatir, Share Folder
            days_to_expire* : string /^[0-9]+$/
            emails* : Emails array string '{'format': 'email'}'
            folder_id* : Folder identifier string /^[0-9]+$/
            Les paramètres de type Object peuvent être envoyés sous forme aplatie, avec les clés et les sous-clés séparées par le caractère /, par exemple : {"element": {"child": "value"}} peut être envoyé ainsi : {"element/child": "value"}.

J'imagine que le commentaire sur "les paramètres de type Object" est lié à emails qui est un "array string"...?

Le schéma donne un affichage cabalistique: <pre> Share a folder to external users : /filr-rest/cd44/share-folder Corps de la requête application/json : object, désaplatir, Share Folder days_to_expire* : string /^[0-9]+$/ emails* : Emails array string '{'format': 'email'}' folder_id* : Folder identifier string /^[0-9]+$/ Les paramètres de type Object peuvent être envoyés sous forme aplatie, avec les clés et les sous-clés séparées par le caractère /, par exemple : {"element": {"child": "value"}} peut être envoyé ainsi : {"element/child": "value"}. </pre> J'imagine que le commentaire sur "les paramètres de type Object" est lié à emails qui est un "array string"...?
ecazenave commented 2023-01-17 14:31:09 +01:00
Author
Owner
Copy Link

C'est à cause du unflatten : https://dev.entrouvert.org/issues/73500

J'ai ajouté un exemple de requête pour éclaircir les choses.

C'est à cause du unflatten : https://dev.entrouvert.org/issues/73500 J'ai ajouté un exemple de requête pour éclaircir les choses.
Ghost marked this conversation as resolved
ecazenave force-pushed wip/73226-filr-rest from c66d57f6b3 to 794edc1ce0 2023-01-17 13:55:35 +01:00 Compare
ecazenave changed title from WIP: filr_rest: start connector (#73226) to filr_rest: start connector (#73226) 2023-01-17 14:31:16 +01:00
Ghost approved these changes 2023-01-18 14:23:34 +01:00
ecazenave force-pushed wip/73226-filr-rest from 794edc1ce0 to 21ac84b4c2 2023-01-18 14:30:36 +01:00 Compare
ecazenave closed this pull request 2023-01-18 14:31:19 +01:00
Some checks reported errors
gitea-wip/passerelle/pipeline/pr-main Build started...
Details
gitea/passerelle/pipeline/head Something is wrong with the build of this commit
Details

Pull request closed

Please reopen this pull request to perform a merge.
Sign in to join this conversation.
Reviewers
No reviewers
Ghost
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#34
No description provided.
Delete Branch "wip/73226-filr-rest"

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?