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

Issue with Untrusted to Trusted Workspaces in Jupyter Notebooks #20367

Open
eleanorjboyd opened this issue Nov 29, 2022 · 2 comments
Open

Issue with Untrusted to Trusted Workspaces in Jupyter Notebooks #20367

eleanorjboyd opened this issue Nov 29, 2022 · 2 comments
Assignees
Labels
bug Issue identified by VS Code Team member as probable bug triage-needed Needs assignment to the proper sub-team

Comments

@eleanorjboyd
Copy link
Member

Currently the python extension is stalling when attempting to execute functionality following the change from a untrusted to trusted workspace. When a file is originally opened in an untrusted workspace, then switched to trusted, the follow jupyter notebook operations performed by the user will stall and not complete. Dev tools will show the following error about the client not be available.

Steps to reproduce:

  1. Open new workspace, click do not trust
  2. Make a new Jupyter notebook
  3. Attempt to run code in the notebook
  4. Check in dev tools, an error should have appeared
  5. VS Code will now stall and be stuck on that run command

We have also determined this issue does not happen when you originally check trust when opening a new workspace. If you make it a trusted workspace from the beginning, then try and run code in a jupyter notebook, the code will return and no error will appear in the dev tools windows.

From the code on the python side, we are able to know that the user has switched to a trusted workspace, and doing this action does not cause a problem. The problem arises when attempting to run a notebook and the client error surfaces.

Screenshot 2022-11-23 at 11 34 34 AM

Screenshot 2022-11-22 at 3 07 24 PM

@VSCodeTriageBot VSCodeTriageBot added the triage-needed Needs assignment to the proper sub-team label Nov 29, 2022
@Tyriar Tyriar assigned rebornix and unassigned Tyriar Nov 29, 2022
@VSCodeTriageBot VSCodeTriageBot removed the triage-needed Needs assignment to the proper sub-team label Nov 29, 2022
@sbatten
Copy link
Member

sbatten commented Nov 29, 2022

The error seems to be coming from the python side when trying to talk to the language server client. I wonder if the client is started too close to the transition from workspace trust.

this.workspace.onDidGrantWorkspaceTrust(() => {

@sbatten
Copy link
Member

sbatten commented Nov 30, 2022

I verified that this is not a regression in either the extension or core, but it seems like the Python extension either tries to access its client too early or our notification of workspace trust to extensions is too early. I'd need to confirm in the extension first.

@rebornix rebornix removed their assignment Dec 1, 2022
@rzhao271 rzhao271 added the bug Issue identified by VS Code Team member as probable bug label Dec 9, 2022
@karrtikr karrtikr transferred this issue from microsoft/vscode Dec 12, 2022
@karrtikr karrtikr added the triage-needed Needs assignment to the proper sub-team label Dec 12, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Issue identified by VS Code Team member as probable bug triage-needed Needs assignment to the proper sub-team
Projects
None yet
Development

No branches or pull requests

8 participants