[issue2528] Change os.access to check ACLs under Windows

Eryk Sun added the comment: With increasing use of os.access() in shutil and tempfile, it would be nice to have a real implementation of os.access() for Windows. Instead of manually evaluating the security of the file/directory, as issue2528.2.patch attempts to do, I'd rather just open the

[issue2528] Change os.access to check ACLs under Windows

Changes by Mark Lawrence : -- nosy: -BreamoreBoy ___ Python tracker ___ ___ Python-bugs-list mailing list Unsubscribe: https://mail.p

[issue2528] Change os.access to check ACLs under Windows

Changes by Piotr Dobrogost : -- nosy: +piotr.dobrogost ___ Python tracker ___ ___ Python-bugs-list mailing list Unsubscribe: https://m

[issue2528] Change os.access to check ACLs under Windows

Changes by Paul Moore : -- nosy: +paul.moore ___ Python tracker ___ ___ Python-bugs-list mailing list Unsubscribe: https://mail.python

[issue2528] Change os.access to check ACLs under Windows

Tim Golden added the comment: Thanks for the very thorough review. This isn't going to make it into 3.5, but I'll rework it in the light of your comments and see if people are happy with it in the optional argument variation. -- ___ Python tracker

[issue2528] Change os.access to check ACLs under Windows

eryksun added the comment: In msg243815 you asked me to look over this patch. I hope this helps. For GetFileSecurity you need to also request LABEL_SECURITY_INFORMATION. To test this in Vista+, use a file in the root directory of the system drive. This will inherit a high integrity level w/ no

[issue2528] Change os.access to check ACLs under Windows

Changes by Serhiy Storchaka : -- nosy: +serhiy.storchaka ___ Python tracker ___ ___ Python-bugs-list mailing list Unsubscribe: https:/

[issue2528] Change os.access to check ACLs under Windows

Mark Lawrence added the comment: The solution proposed here could help resolve #22107. -- nosy: +BreamoreBoy ___ Python tracker ___ ___

[issue2528] Change os.access to check ACLs under Windows

Changes by Mark Lawrence : -- nosy: -BreamoreBoy ___ Python tracker ___ ___ Python-bugs-list mailing list Unsubscribe: https://mail.p

[issue2528] Change os.access to check ACLs under Windows

Changes by Tim Golden : Removed file: http://bugs.python.org/file9919/os_access-r62091.patch ___ Python tracker ___ ___ Python-bugs-list mailin

[issue2528] Change os.access to check ACLs under Windows

Tim Golden added the comment: ... and to answer Amaury's question in msg109871 it creates a reasonable consistency between the results of os.access and the user's actual ability to read / write a file. eg, you might have no permissions whatsoever on the file but as long as it wasn't read-only,

[issue2528] Change os.access to check ACLs under Windows

Tim Golden added the comment: Here's an updated patch against trunk with tests & doc changes -- status: languishing -> open Added file: http://bugs.python.org/file31165/issue2528.2.patch ___ Python tracker _

[issue2528] Change os.access to check ACLs under Windows

Christian Heimes added the comment: Do you want to provide an updated patch for 3.4? -- nosy: +christian.heimes status: open -> languishing versions: +Python 3.4 -Python 3.2 ___ Python tracker __

[issue2528] Change os.access to check ACLs under Windows

Changes by Tim Golden : -- assignee: -> tim.golden ___ Python tracker ___ ___ Python-bugs-list mailing list Unsubscribe: http://mail.

[issue2528] Change os.access to check ACLs under Windows

Mark Lawrence added the comment: Tim, do you want more time to think about this, could we close as "won't fix" or what? -- ___ Python tracker ___ ___

[issue2528] Change os.access to check ACLs under Windows

Tim Golden added the comment: Although I'm the implementer of the patch (the concept was discussed way back on c.l.py after a naive poster's original request) I'm probably +0 myself. It's an attempt to replace os.access' next-to-useless behaviour on Windows with something which at least uses cur

[issue2528] Change os.access to check ACLs under Windows

Amaury Forgeot d'Arc added the comment: But what are the benefits of this change? -- nosy: +amaury.forgeotdarc ___ Python tracker ___

[issue2528] Change os.access to check ACLs under Windows

Mark Lawrence added the comment: A quick look tells me that the patch seems clean. However it involves changes to posixmodule.c and I don't (yet) want to get involved with doing builds, so could someone else please give this a try. -- nosy: +BreamoreBoy versions: -Python 2.7 __

[issue2528] Change os.access to check ACLs under Windows

Changes by Daniel Diniz : -- components: +Windows priority: -> normal stage: -> patch review type: -> feature request versions: +Python 2.7, Python 3.2 -Python 2.6 ___ Python tracker _

[issue2528] Change os.access to check ACLs under Windows

New submission from Tim Golden <[EMAIL PROTECTED]>: At present, os.access under Windows simply calls GetFileAttributes to determine the readonly attribute (ignoring directories). The patch attached combines this with the use of the AccessCheck API to compare the user's permissions with those requ