Skip to content
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

Made resolveExternalEntities depend of workspace trust #545

Merged
merged 1 commit into from
Jul 8, 2021

Conversation

AlexXuChen
Copy link
Contributor

Changed the resolveExternalEntities setting to depend on workspace trust, where it uses user settings in trusted workspaces, and is false in untrusted workspaces.

Closes #537

Signed-off-by: Alexander Chen [email protected]

@AlexXuChen AlexXuChen requested a review from datho7561 July 7, 2021 18:39
src/settings/settings.ts Outdated Show resolved Hide resolved
src/settings/settings.ts Outdated Show resolved Hide resolved
src/settings/settings.ts Outdated Show resolved Hide resolved
@AlexXuChen AlexXuChen merged commit ca210a2 into redhat-developer:master Jul 8, 2021
@AlexXuChen AlexXuChen deleted the issue537 branch July 8, 2021 14:56
@GitMensch
Copy link
Contributor

The change is good, but I think this really needs to be documented, at least on the setting itself.

@AlexXuChen
Copy link
Contributor Author

Documentation: #546

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.

have an option to make "resolve external entities" depending on workspace trust
3 participants