"WARN: Unable to run doctor. Make sure isHttpEnabled is set to true." and it is impossible to set to true
#7072
Comments
|
Thanks for reporting, it should be possible to set it in the invocation of metals binary, no? Either way it seems redundant, maybe we should just start the server in that case |
|
thanks a lot, but how to set the invocation of metaLS binary? there is no launching command-line option |
|
What editor are you using? That should be possible to do in some way for every editor. In VS Code it's done by modifying metals.serverProperties, in Zed it's done in the config via: |
|
The IDE is IntelliJ IDEA + LSP4IJ plugin, these are my configurations: it is from the template at https://github.com/redhat-developer/lsp4ij/blob/6913946576da78a75131ef19e8af0efb37221526/src/main/resources/templates/metals/settings.json tho the schema file is missing, so I have no idea if it is up-to-date |
|
You would need to do currently: that is my best guess |
|
thanks a lot @tgodzik this does nothing to the error: still got the same warning (setting in LSP4IJ attached).
is -Dmetals.http defined somewhere in scalameta sourcecode? I'll need to generate a schema file from it |
|
It's defined in MetalsServerConfig While serverProperties itself are defined in UserConfiguration From what I checked again isHttpEnabled only needs to be in serverProperties, but it can also be added to initialization options sent from the client under, though I have no idea how it works in the Intellij LSP client. |
|
I will add an option to start the server despite the option not being set later on today. |
tgodzik
added
improvement
This will show up after indexing and users can start the server themselves or ignore it for the session. Should help with scalameta#7072
|
Btw. does the http server start? I noticed that we would try to access it before it's actually started and print the warning. Fixing it here #7084 |
Describe the bug
the log contains the following information:
full log:
since Metals has no commandline option, it is impossible to set it to true
Expected behavior
need an option to set it to true
Operating system
Linux
Editor/Extension
Other
Version of Metals
1.4.2
Extra context or search terms
No response
The text was updated successfully, but these errors were encountered: