Skip to content

fix #732 - add NoZone to allowed zones #841

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Closed
wants to merge 1 commit into from

Conversation

himbeles
Copy link

@himbeles himbeles commented Apr 5, 2019

What does this implement/fix? Explain your changes.

In addition to 'MyComputer', add 'NoZone' to the list of allowed zones.
System.Security.SecurityZone.NoZone is added to the allowed zones in order to prevent "File is blocked (NTFS Security)" error on certain Windows and mac installations.
Fixes issue #732.
...

Does this close any currently open issues?

fixes issue 732
...

Any other comments?

only tested locally on a windows 7 installation
...

Checklist

Check all those that are applicable and complete.

  • Make sure to include one or more tests for your change
  • If an enhancement PR, please create docs and at best an example
  • Add yourself to AUTHORS
  • Updated the CHANGELOG

Add System.Security.SecurityZone.NoZone to the allowed zones in order to prevent "File is blocked (NTFS Security)" error on certain Windows and mac installations. Fixes issue pythonnet#732.
@filmor
Copy link
Member

filmor commented Apr 5, 2019

Thank you very much for your contribution, but I think I'll just revert #655 entirely, the only thing it "fixes" was a confusing error message while it created problems in various places :)

@himbeles
Copy link
Author

himbeles commented Apr 5, 2019 via email

@den-run-ai
Copy link
Contributor

@filmor maybe providing a warning messages if assembly loading fails due to security zone?

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

Successfully merging this pull request may close these issues.

3 participants