-
Notifications
You must be signed in to change notification settings - Fork 758
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
debug: connecting to dlv-dap results in connection error #1270
Comments
This happens when the server is not up and ready to listen. Do we have a sufficient delay in the factory? I think there used to be a start-up delay in the old adapter as well. |
The error message is coming from the editor. I don't know how to control it. |
There was a delay but it was not sufficient. Once @hyangah's change for redirecting stdout and stderr from dlv dap is in, I have a change that will wait for the "DAP serving listening at" message before returning the server info to vscode. |
Change https://golang.org/cl/300071 mentions this issue: |
Change https://golang.org/cl/300609 mentions this issue: |
…tory When we start dlv dap, we want to make sure it has started before sending the info back to vscode so it will be able to connect. Wait for the 'DAP server listening at' message before returning the host. Display the error for the user otherwise. Fixes #1270 Change-Id: Id710b67ceaa87b1f6dff84d8108ac61dfbe15707 Reviewed-on: https://go-review.googlesource.com/c/vscode-go/+/300071 Trust: Suzy Mueller <[email protected]> Run-TryBot: Suzy Mueller <[email protected]> TryBot-Result: kokoro <[email protected]> Reviewed-by: Hyang-Ah Hana Kim <[email protected]> Reviewed-on: https://go-review.googlesource.com/c/vscode-go/+/300609 Trust: Hyang-Ah Hana Kim <[email protected]>
What version of Go, VS Code & VS Code Go extension are you using?
go version
to get version of Go from the VS Code integrated terminal.code -v
orcode-insiders -v
to get version of VS Code or VS Code Insiders.Describe the bug
Steps to reproduce the behavior:
This appears to be a timing related issue, since it does not happen often. This needs a better error message and/or a second attempt to connect.
The text was updated successfully, but these errors were encountered: