Skip to content
This repository has been archived by the owner on Nov 25, 2020. It is now read-only.

Improved permission detection when using ftp backend #429

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

aseques
Copy link

@aseques aseques commented Jan 23, 2014

Added the same improvements when using the ftp backend that allow better permission checking when on POSIX systems.
Basically adapted the patch from here: http://pyd.io/f/topic/solved-linux-acl-folder-not-writeable/

If applicable, copyright goes to the projec..

@cdujeu
Copy link
Member

cdujeu commented Apr 2, 2014

@aseques, the real path here is referring to something like ftp:// ... can we really check POSIX on a remote server via FTP?

@aseques
Copy link
Author

aseques commented Apr 2, 2014

The issue in my case is that a user (user1) connects to an ftp and has read/write access to a subfolder owned by another user (user2)

user1
---------user2
----------------- files

Unless the check is done via posix_access, the is_writable() will return false when checking permissions on user2 folder.

@cdujeu
Copy link
Member

cdujeu commented Sep 21, 2016

@c12simple what do you think?

@cdujeu cdujeu added this to the 6.5.4 => 7.0.0 milestone Sep 21, 2016
@cdujeu cdujeu modified the milestones: 6.5.3, 6.5.4 => 7.0.0 Sep 29, 2016
@c12simple
Copy link
Member

Hello @aseques
Could you please tell me how to reproduce this issue?
FTP service: the OS of FTP, ftp server program , permission setting. .....

@c12simple
Copy link
Member

Hello @aseques
Please contact me when you have time.
THank you very much.

@aseques
Copy link
Author

aseques commented Oct 3, 2016

Sorry I can't remember the details, it's been a lot of time (I even changed jobs), but it basically it's in the issue. I was using proftpd with the database backend (mysql).

@cdujeu cdujeu modified the milestones: 6.5.4 => 7.0.0, 6.5.3, 6.5.4, 6.5.5 => 7.0.0 Oct 4, 2016
@c12simple
Copy link
Member

Hello @aseques
Thank you, that's enough for me to reproduce by using proftpd.

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants