Skip to content

Closing "A request has failed." notification opens output panel #10407

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
nyanpasu64 opened this issue Sep 30, 2021 · 5 comments · Fixed by #10421
Closed

Closing "A request has failed." notification opens output panel #10407

nyanpasu64 opened this issue Sep 30, 2021 · 5 comments · Fixed by #10421

Comments

@nyanpasu64
Copy link

In the VS Code plugin, if I close the "A request has failed." with the close button, or it gets automatically closed by a second identical message, it opens the output panel automatically. I expect it to only happen when I press the "Go to output" button.

As to how I'm getting so many crashes that it's becoming a nuisance... well, #10386.

I'm on VS Code plugin version v0.3.763.

@lnicola
Copy link
Member

lnicola commented Oct 1, 2021

Probably a duplicate of microsoft/vscode-languageserver-node#786.

@lnicola lnicola closed this as completed Oct 1, 2021
@nyanpasu64
Copy link
Author

nyanpasu64 commented Oct 1, 2021

How is this a duplicate? That bug was filed in a different repository (and this repository doesn't seem to be a fork of that, and I'm not sure if and how the rust-analyzer extension depends on it), was resolved 2.5 months ago, and the bug is still present in the rust-analyzer extension. When will it be fixed here?

@lnicola lnicola reopened this Oct 2, 2021
@bjorn3
Copy link
Member

bjorn3 commented Oct 2, 2021

and this repository doesn't seem to be a fork of that, and I'm not sure if and how the rust-analyzer extension depends on it

The vscode extension uses vscode-languageserver-node to communicate with the rust-analyzer language server. Vscode doesn't have an LSP implementation itself. All vscode LSP based extensions need to use this package.

https://github.com/rust-analyzer/rust-analyzer/blob/237ea0d34dced3444486931f68e87cd07f52b6a8/editors/code/package.json#L41

was resolved 2.5 months ago, and the bug is still present in the rust-analyzer extension.

Maybe updating the dependency version will work? Updating could be non-trivial though. We are currently using 7.1.0-next.5. The main branch on which it should be fixed is 8.0.0-next.2: https://github.com/microsoft/vscode-languageserver-node/blob/10b56de150ad67c3c330da8e2df53ebf2cf347c4/client/package.json#L4

@lnicola
Copy link
Member

lnicola commented Oct 2, 2021

Yeah, sorry about it. The bug was in the Code LSP client, but I missed the new release (I had the page in an open tab). Filed #10421 for it.

@bors bors bot closed this as completed in c88277f Oct 2, 2021
@nyanpasu64
Copy link
Author

Thanks for the fix!

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 a pull request may close this issue.

3 participants