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

2021-03-11 Thread Eryk Sun
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

2015-05-23 Thread Paul Moore
Changes by Paul Moore p.f.mo...@gmail.com: -- nosy: +paul.moore ___ Python tracker rep...@bugs.python.org http://bugs.python.org/issue2528 ___ ___ Python-bugs-list

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

2015-05-23 Thread eryksun
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/

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

2015-05-23 Thread Tim Golden
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

2015-02-17 Thread Mark Lawrence
Mark Lawrence added the comment: The solution proposed here could help resolve #22107. -- nosy: +BreamoreBoy ___ Python tracker rep...@bugs.python.org http://bugs.python.org/issue2528 ___

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

2015-02-17 Thread Serhiy Storchaka
Changes by Serhiy Storchaka storch...@gmail.com: -- nosy: +serhiy.storchaka ___ Python tracker rep...@bugs.python.org http://bugs.python.org/issue2528 ___ ___

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

2014-02-03 Thread Mark Lawrence
Changes by Mark Lawrence breamore...@yahoo.co.uk: -- nosy: -BreamoreBoy ___ Python tracker rep...@bugs.python.org http://bugs.python.org/issue2528 ___ ___

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

2013-08-05 Thread Tim Golden
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 rep...@bugs.python.org

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

2013-08-05 Thread Tim Golden
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

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

2013-08-05 Thread Tim Golden
Changes by Tim Golden m...@timgolden.me.uk: Removed file: http://bugs.python.org/file9919/os_access-r62091.patch ___ Python tracker rep...@bugs.python.org http://bugs.python.org/issue2528 ___

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

2013-07-09 Thread Christian Heimes
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 rep...@bugs.python.org http://bugs.python.org/issue2528

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

2010-08-25 Thread Tim Golden
Changes by Tim Golden m...@timgolden.me.uk: -- assignee: - tim.golden ___ Python tracker rep...@bugs.python.org http://bugs.python.org/issue2528 ___ ___ Python-bugs-list

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

2010-07-19 Thread Mark Lawrence
Mark Lawrence breamore...@yahoo.co.uk added the comment: Tim, do you want more time to think about this, could we close as won't fix or what? -- ___ Python tracker rep...@bugs.python.org http://bugs.python.org/issue2528

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

2010-07-10 Thread Mark Lawrence
Mark Lawrence breamore...@yahoo.co.uk 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

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

2010-07-10 Thread Amaury Forgeot d'Arc
Amaury Forgeot d'Arc amaur...@gmail.com added the comment: But what are the benefits of this change? -- nosy: +amaury.forgeotdarc ___ Python tracker rep...@bugs.python.org http://bugs.python.org/issue2528 ___

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

2010-07-10 Thread Tim Golden
Tim Golden m...@timgolden.me.uk 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

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

2009-05-16 Thread Daniel Diniz
Changes by Daniel Diniz aja...@gmail.com: -- components: +Windows priority: - normal stage: - patch review type: - feature request versions: +Python 2.7, Python 3.2 -Python 2.6 ___ Python tracker rep...@bugs.python.org

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

2008-04-01 Thread Tim Golden
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