-
Notifications
You must be signed in to change notification settings - Fork 374
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
jdt.ls 1.23.0 issues invalid reverse request "workspace/executeClientCommand' #2633
Comments
Maybe we need to add a client capability here. Definition of workspace/executeClientCommand is here: We use it to trigger a command execution from server to client. |
Yup. The server requests a command be executed on the client side without any check as to whether the client supports this : Lines 429 to 435 in f4d1a4a
A "command" is just supposed to represent some logic/task that the client can carry out simply by referring to it through an identifier. I would think this isn't harmful as the client should just fail to carry out the unknown request, but maybe certain clients can't gracefully recover so we should handle definitely handle it. I think it would have to be an There are other locations where we call Lines 213 to 214 in f4d1a4a
java.completion.onDidSelect which is not something many clients are as likely to be calling.
Line 122 in f4d1a4a
java/organizeImports . There are other ways of triggering organize imports thoug so this is also less likely to happen.
|
I'm interested in working on this |
Since updating to 1.23.0, I'm seeing the following invalid message being sent to ycmd:
The
workspace/executeClientCommand
method does not exist in the LSP: https://microsoft.github.io/language-server-protocol/specifications/lsp/3.17/specification/this is leading to an exception trace.
The text was updated successfully, but these errors were encountered: