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

"Learn More" button unexpectedly orphans unsaved editors when reopening workspaces on untrusted UNC hosts #181956

Closed
ThubLives opened this issue May 9, 2023 · 1 comment
Assignees
Labels
bug Issue identified by VS Code Team member as probable bug insiders-released Patch has been released in VS Code Insiders on-testplan unc
Milestone

Comments

@ThubLives
Copy link

Does this issue occur when all extensions are disabled?: Yes

Version: 1.78.1 (system setup)
Commit: 6a995c4
Date: 2023-05-04T09:48:08.683Z
Electron: 22.5.1
Chromium: 108.0.5359.215
Node.js: 16.17.1
V8: 10.8.168.25-electron.0
OS: Windows_NT x64 10.0.19044
Sandboxed: No

Steps to Reproduce:

  1. Open a workspace at a UNC path that isn't set in security.allowedUNCHosts.
  2. Click "Yes" when prompted to temporarily allow working from that host.
  3. Make changes to one or more files without saving them.
  4. Restart VSCode without closing the workspace (may require specific settings to automatically re-open the previous workspace).
  5. This time, click the "Learn More" button when prompted to allow working from that host.

Note that any unsaved editors remain, but the rest of the workspace is not loaded as though the user clicked the "No" button. I'd argue that "Learn More" doesn't constitute a decision from the user and shouldn't proceed as though it was. Depending on the nature of the changes, this may cause a problem for safely saving those changes because if the user tries to re-open the workspace, they are asked to either save or close these orphaned editors.

Perhaps a link among the text in the dialogue box would be more appropriate than a button which closes the box.

@bpasero bpasero added bug Issue identified by VS Code Team member as probable bug and removed triage-needed labels May 10, 2023
@bpasero bpasero added this to the May 2023 milestone May 10, 2023
@bpasero
Copy link
Member

bpasero commented May 10, 2023

I agree that the dialog should not close and/or make a decision when you click "Learn More". Unfortunately the dialog closes when you press that button, so best I can do is to open it again asking the same question.

@bpasero bpasero added the unc label May 10, 2023
@vscodenpa vscodenpa added unreleased Patch has not yet been released in VS Code Insiders insiders-released Patch has been released in VS Code Insiders and removed unreleased Patch has not yet been released in VS Code Insiders labels May 10, 2023
@github-actions github-actions bot locked and limited conversation to collaborators Jun 24, 2023
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
bug Issue identified by VS Code Team member as probable bug insiders-released Patch has been released in VS Code Insiders on-testplan unc
Projects
None yet
Development

No branches or pull requests

3 participants