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

x/tools/gopls: makes http requests to random websites while editing module directive #43187

Closed
josharian opened this issue Dec 14, 2020 · 2 comments
Closed

Comments

@josharian
Copy link
Contributor

@josharian josharian commented Dec 14, 2020

In an attempt to make a small reproducer for #43186, I created a go.mod that started:

module vanilla.xyz

After saving, I selected the word vanilla and started typing "chocolate" as a replacement. As I typed, gopls started attempting to retrieve results from intermediate domains choco.xyz, chocola.xyz, chocolat.xyz, despite me not hitting save. Sample alert from Little Snitch:

Screen Shot 2020-12-14 at 2 36 33 PM

The partial module import path seemed to get cached, because well after I finished typing an hit save, I still got this error:

Screen Shot 2020-12-14 at 2 37 22 PM

@gopherbot gopherbot added this to the Unreleased milestone Dec 14, 2020
@stamblerre
Copy link
Contributor

@stamblerre stamblerre commented Dec 15, 2020

gopls currently tries to reload package metadata as you modify import statements in your file, which is why you saw these requests. We have made some changes in recent weeks to default to GOPROXY=off--can you try the latest gopls pre-release and confirm that it fixes the issue? You can download it by running GO111MODULE=on go get golang.org/x/tools/gopls@v0.6.0-pre.1.

@stamblerre stamblerre added this to Needs Triage in vscode-go: gopls by default via automation Dec 16, 2020
@stamblerre stamblerre moved this from Needs Triage to Waiting for Info in vscode-go: gopls by default Dec 16, 2020
@stamblerre stamblerre moved this from Waiting for Info to Non-critical in vscode-go: gopls by default Jan 8, 2021
@stamblerre stamblerre moved this from Non-critical to Needs Triage in vscode-go: gopls by default Jan 8, 2021
@stamblerre stamblerre removed this from Needs Decision in vscode-go: gopls by default Jan 8, 2021
@stamblerre stamblerre modified the milestones: gopls/v0.6.3, gopls/v1.0.0 Jan 14, 2021
@gopherbot
Copy link

@gopherbot gopherbot commented Jan 15, 2021

Timed out in state WaitingForInfo. Closing.

(I am just a bot, though. Please speak up if this is a mistake or you have the requested information.)

@gopherbot gopherbot closed this Jan 15, 2021
@stamblerre stamblerre removed this from the gopls/v1.0.0 milestone Jan 15, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Linked pull requests

Successfully merging a pull request may close this issue.

None yet
3 participants
You can’t perform that action at this time.