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

server is not initialized correctly without configuration call #601

Closed
gorkem opened this issue Mar 25, 2018 · 5 comments
Closed

server is not initialized correctly without configuration call #601

gorkem opened this issue Mar 25, 2018 · 5 comments
Assignees
Labels

Comments

@gorkem
Copy link
Contributor

gorkem commented Mar 25, 2018

Server is not intialized correctly if the client does not send didChangeConfiguration. It is possible that a client does not support settings. Also LSP does not really define an order for the didChangeConfiguration notification. Instead of depending on didChangeConfigration we should use the initialized notification.

@gorkem
Copy link
Contributor Author

gorkem commented Mar 25, 2018

No. If the client reports it supports dynamic registration without this part executing server basically reports that it supports nothing on its capabilities

@puremourning
Copy link
Contributor

I can confirm that i had to put in the didChangeConfiguration event to make jdt.ls start up properly:

https://github.com/Valloric/ycmd/blob/master/ycmd/completers/language_server/language_server_completer.py#L1257

@fbricon
Copy link
Contributor

fbricon commented Mar 25, 2018

I see, we assumed didChangeConfiguration requests were sent on startup if dynamic registration is supported, because of vscode's behavior

@puremourning
Copy link
Contributor

I don't think it is a big deal. We'll almost certainly need to support the config stuff at some point anyway.

gorkem added a commit that referenced this issue Mar 26, 2018
Moves the dynamic registration of the server capabilities
to initialized notification

fixes #601

Signed-off-by: Gorkem Ercan <[email protected]>
@fbricon fbricon added this to the End March 2018 milestone Mar 26, 2018
gorkem added a commit to gorkem/eclipse.jdt.ls that referenced this issue Mar 26, 2018
Moves the dynamic registration of the server capabilities
to initialized notification

fixes eclipse-jdtls#601

Signed-off-by: Gorkem Ercan <[email protected]>
gorkem added a commit to gorkem/eclipse.jdt.ls that referenced this issue Mar 26, 2018
Moves the dynamic registration of the server capabilities
to initialized notification

fixes eclipse-jdtls#601

Signed-off-by: Gorkem Ercan <[email protected]>
@gorkem gorkem closed this as completed in a3fb99c Mar 26, 2018
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging a pull request may close this issue.

3 participants