Steve Dower added the comment:

+Ned

Could we get a doc patch into 3.6 marking this class as deprecated? It appears 
like the importlib docs are the only ones that refer to the class, and none of 
the docs describe the functionality or indicate that it is enabled by default.

I could also pitch this as a security vulnerability and push for removing the 
default .append() right now? Since we wouldn't remove the class itself, 
restoring the previous behavior just requires inserting it into meta_path 
again. And Alexey is right that it actually allows a non-admin user to shadow 
any non-builtin module.

Looking at the latest pywin32 installer, they actually *remove* the keys they 
used to add here because they cause problems. So I think we're fairly safe to 
disable the finder by default and deprecate it into the future.

----------
nosy: +ned.deily

_______________________________________
Python tracker <rep...@bugs.python.org>
<http://bugs.python.org/issue28896>
_______________________________________
_______________________________________________
Python-bugs-list mailing list
Unsubscribe: 
https://mail.python.org/mailman/options/python-bugs-list/archive%40mail-archive.com

Reply via email to