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/pkgsite: unable to fetch packages from module proxy (go.buf.build) #48952

Closed
lrewega opened this issue Oct 13, 2021 · 8 comments
Closed

x/pkgsite: unable to fetch packages from module proxy (go.buf.build) #48952

lrewega opened this issue Oct 13, 2021 · 8 comments
Labels
FrozenDueToAge NeedsFix The path to resolution is known, but the work has not been done. pkgsite

Comments

@lrewega
Copy link

lrewega commented Oct 13, 2021

What is the URL of the page with the issue?

https://pkg.go.dev/go.buf.build/library/go-grpc/envoyproxy/protoc-gen-validate/validate

What is your user agent?

Mozilla/5.0 (Macintosh; Intel Mac OS X 10_15_7) AppleWebKit/537.36 (KHTML, like Gecko) Chrome/94.0.4606.71 Safari/537.36

Screenshot

Screen Shot 2021-10-13 at 4 36 25 PM

Clicking the "Request" button leads to:

Screen Shot 2021-10-13 at 4 37 24 PM

What did you do?

Attempted to view documentation for any package available through go.buf.build, a module proxy.

What did you expect to see?

Generated documentation for the package, e.g.:

https://godocs.io/go.buf.build/library/go-grpc/envoyproxy/protoc-gen-validate/validate

What did you see instead?

The "Oops!" page as seen in the above screenshots.


I've tried following each of the linked instructions to fetch the package but none appear to succeed.

godocs.io seems to handle this fine, so I am not sure if the problem is with pkgsite or go.buf.build. If it's the latter, any guidance on what's incorrect would be much appreciated.

@gopherbot gopherbot added this to the Unreleased milestone Oct 13, 2021
@jamalc jamalc added the NeedsInvestigation Someone must examine and confirm this is a valid issue and not a duplicate of an existing one. label Oct 13, 2021
@jamalc jamalc modified the milestones: Unreleased, pkgsite/unplanned Oct 13, 2021
@jamalc
Copy link

jamalc commented Oct 13, 2021

Following CL 245918 pkg.go.dev does not process modules with zero commit times. The proxy does support them, maybe this is something we should revisit? I'm not sure of the exact motivation behind the change.

cc/ @jba @julieqiu

@seankhliao
Copy link
Member

go.buf.build would also need to include a license for the generated code

@jba jba self-assigned this Oct 14, 2021
@jba
Copy link
Contributor

jba commented Oct 14, 2021

I now believe this is our bug. We should handle zero commit times.

@jba jba added NeedsFix The path to resolution is known, but the work has not been done. and removed NeedsInvestigation Someone must examine and confirm this is a valid issue and not a duplicate of an existing one. labels Oct 14, 2021
@gopherbot
Copy link

Change https://golang.org/cl/356129 mentions this issue: internal/{postgres,frontend}: support modules with zero commit times

@lrewega
Copy link
Author

lrewega commented Oct 18, 2021

@jba has your fix been deployed, or is there perhaps another issue? I'm still getting "could not be found" messages for the packages in the issue summary. If there is not a valid license, I believe I should see a message to that effect.

@jba
Copy link
Contributor

jba commented Oct 18, 2021

The fix hasn't been deployed yet. I'll ping this issue when the problem is resolved. (As per standard but confusing GitHub etiquette, the issue gets closed when the fixing PR is submitted, not when the problem is actually fixed.)

@lrewega
Copy link
Author

lrewega commented Oct 18, 2021

@jba great, thanks! I appreciate it.

@jba
Copy link
Contributor

jba commented Oct 20, 2021

This is fixed. (You still have a license problem.)

@rsc rsc unassigned jba Jun 23, 2022
@golang golang locked and limited conversation to collaborators Jun 23, 2023
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
FrozenDueToAge NeedsFix The path to resolution is known, but the work has not been done. pkgsite
Projects
None yet
Development

No branches or pull requests

5 participants