-
-
Notifications
You must be signed in to change notification settings - Fork 895
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
Error in go-langserver: No Content-Length header #412
Comments
Shouldn't this be on the |
@vibhavp IMO, it's related to the data parsing in common part of |
Yup, but if it really was an erroneous parser, there would have a dozen more reports about other language packages breaking as well. Could you paste the contents of the |
Thanks for the reply. I pasted stderr output above. Another thing I can confirm is it's related to the changes in Go 1.11. For 1.10, it works fine. I read the release notes of Go 1.11, it seems not a bug. May need other packages to be compatible with the new release. Honestly I am not certain. So report the issue in both |
Could be the |
I did the tests and attached the screen and log file. |
Close it since the issue has been fixed in sourcegraph/go-langserver@d0a69e9. |
@seagle0128 I'm also using emacs with LSP-go enabled. After this commit sourcegraph/go-langserver@d0a69e9, I do get get the panic error, but I still get the |
@bittopaz I believe you got the answer on sourcegraph/go-langserver#317 (comment). I think it's not a real issue, while is a ungraceful error handling. BTW, I am using Go 1.10.x with happiness. go-langserver is not stable on Go 1.11. |
@seagle0128 thanks for the reply. |
@bittopaz Reinstall go get -u github.com/sourcegraph/go-langserver
go clean -cache
rm -rf $GOPATH/pkg/ |
@seagle0128 I just tried these commands, but still no luck. That's weird, everything works when I was originally on go 1.10.3. But now, when I switch back to 1.10.3, it stops working. This bugs me for a whole week... |
@bittopaz What doesn't work? autocompletion or just this error? BTW, you should clean old |
I met the same problem almost at the same time as you do, so I subscribe from the very beginning. And now the only error I still get is this one:
I used the command in your last comment, and verified the binary modify time, I believe that's a reinstall. Anyways thanks again, and I'll do more debug on this error. |
IMO, this is not an error, just like a warning. It's just in the |
It is in message buffer, that is why it is annoying |
File an issue for |
@seagle0128 I wasn't sure if it is a go-langserver issue, since the related code did not change for quite a while. I turned to eglot, and now I don't see the errors in the |
@bittopaz OK. I guess BTW, how do think about |
@seagle0128 I guess I would answer your question after a week or two, since I'm just at the started to use it. 😄 But it is really easy to set up, and provide many out of box features. |
@bittopaz Waiting for your update 😄 |
Today, I upgraded
Golang
to 1.11.0 from 1.10.3. And I updatedgo-langserver
. Then I got this error while using auto-completion.Backtrace:
I enabled
-trace
flag and get the output oflsp-go stderr
buffer below.The text was updated successfully, but these errors were encountered: