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

cmis: search first existing parent folder from the end of the path (#83682) #409

Merged
bdauvergne merged 2 commits from wip/83682-cmis-creer-les-repertoires-paren into main 2023-11-27 10:31:36 +01:00
Conversation 2 Commits 2 Files Changed 2 +111 -101
bdauvergne commented 2023-11-17 19:48:31 +01:00
Owner
Copy Link
No description provided.
bdauvergne added 1 commit 2023-11-17 19:48:31 +01:00
gitea/passerelle/pipeline/head There was a failure building this commit Details
d223710ef7 cmis: search first existing parent folder from the end of the path (#83682)
bdauvergne added 1 commit 2023-11-17 20:05:56 +01:00
gitea/passerelle/pipeline/head There was a failure building this commit Details
63d333b501 cmis: produce more precise APIError on cmislib.PermissionDeniedException (#83682)
bdauvergne force-pushed wip/83682-cmis-creer-les-repertoires-paren from 63d333b501 to 4582463d20 2023-11-17 20:08:19 +01:00 Compare
bdauvergne force-pushed wip/83682-cmis-creer-les-repertoires-paren from 4582463d20 to b769478951 2023-11-18 09:49:28 +01:00 Compare
bdauvergne force-pushed wip/83682-cmis-creer-les-repertoires-paren from b769478951 to 4d81811523 2023-11-18 10:33:38 +01:00 Compare
nroche commented 2023-11-20 09:17:41 +01:00
Owner
Copy Link

Merci beaucoup Benjamin.
Maintenant, j'ai un doute sur la pertinence du patch dans mon cas précis.
Je me demande si ça aurait évité les erreurs, parce que si l'accès à un répertoire de plus haut niveau nous est refusé,
alors je ne pense pas qu'on puisse accéder à ses enfants.
https://unix.stackexchange.com/questions/13858/do-the-parent-directorys-permissions-matter-when-accessing-a-subdirectory

you can traverse a directory if and only if you have execute permission on it. (bien que pas lié à la GED)

Je ne peux pas tester : l'erreur ne s'est produite que 3 fois puis a disparue, sans que Toulouse n'ai effectué aucun changement de leur côté.

Merci beaucoup Benjamin. Maintenant, j'ai un doute sur la pertinence du patch dans mon cas précis. Je me demande si ça aurait évité les erreurs, parce que si l'accès à un répertoire de plus haut niveau nous est refusé, alors je ne pense pas qu'on puisse accéder à ses enfants. https://unix.stackexchange.com/questions/13858/do-the-parent-directorys-permissions-matter-when-accessing-a-subdirectory > you can traverse a directory if and only if you have execute permission on it. (bien que pas lié à la GED) Je ne peux pas tester : l'erreur ne s'est produite que 3 fois puis a disparue, sans que Toulouse n'ai effectué aucun changement de leur côté.
bdauvergne commented 2023-11-20 10:16:38 +01:00
Author
Owner
Copy Link

Merci beaucoup Benjamin.
Maintenant, j'ai un doute sur la pertinence du patch dans mon cas précis.
Je me demande si ça aurait évité les erreurs, parce que si l'accès à un répertoire de plus haut niveau nous est refusé,
alors je ne pense pas qu'on puisse accéder à ses enfants.
https://unix.stackexchange.com/questions/13858/do-the-parent-directorys-permissions-matter-when-accessing-a-subdirectory

you can traverse a directory if and only if you have execute permission on it. (bien que pas lié à la GED)

Je ne peux pas tester : l'erreur ne s'est produite que 3 fois puis a disparue, sans que Toulouse n'ai effectué aucun changement de leur côté.

Dans tous les cas mon code est mieux que le code existant, il fait moins d'appels et est plus clair, les tests aussi sont plus clairs; il contien aussi une amélioration au retour des erreurs qui est utile je pense.

> Merci beaucoup Benjamin. > Maintenant, j'ai un doute sur la pertinence du patch dans mon cas précis. > Je me demande si ça aurait évité les erreurs, parce que si l'accès à un répertoire de plus haut niveau nous est refusé, > alors je ne pense pas qu'on puisse accéder à ses enfants. > https://unix.stackexchange.com/questions/13858/do-the-parent-directorys-permissions-matter-when-accessing-a-subdirectory > > you can traverse a directory if and only if you have execute permission on it. (bien que pas lié à la GED) > > Je ne peux pas tester : l'erreur ne s'est produite que 3 fois puis a disparue, sans que Toulouse n'ai effectué aucun changement de leur côté. Dans tous les cas mon code est mieux que le code existant, il fait moins d'appels et est plus clair, les tests aussi sont plus clairs; il contien aussi une amélioration au retour des erreurs qui est utile je pense.
nroche approved these changes 2023-11-22 10:58:17 +01:00
nroche left a comment
Owner
Copy Link

il fait moins d'appels, ... les tests aussi sont plus clairs
Oui.

J'ai testé et ça ne casse rien.

> il fait moins d'appels, ... les tests aussi sont plus clairs Oui. J'ai testé et ça ne casse rien.
bdauvergne merged commit 7de7cd8b3f into main 2023-11-27 10:31:36 +01:00
bdauvergne deleted branch wip/83682-cmis-creer-les-repertoires-paren 2023-11-27 10:31:36 +01:00
Sign in to join this conversation.
Reviewers
No reviewers
nroche
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#409
No description provided.
Delete Branch "wip/83682-cmis-creer-les-repertoires-paren"

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?