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/godoc/dl: store the list of validUsers in a more dynamic way #29898

Open
julieqiu opened this Issue Jan 23, 2019 · 4 comments

Comments

Projects
None yet
4 participants
@julieqiu
Copy link

julieqiu commented Jan 23, 2019

The list of validUsers is hardcoded at https://github.com/golang/tools/blob/master/godoc/dl/dl.go#L317. This means that every time someone is doing a release for a first time, we need to make a CL to add them to this list (so that they can run the release command) and redeploy golang.org.

We may want to store this list in a more dynamic way.

@gopherbot gopherbot added this to the Unreleased milestone Jan 23, 2019

@julieqiu

This comment has been minimized.

Copy link
Author

julieqiu commented Jan 23, 2019

/cc @bradfitz

@bradfitz

This comment has been minimized.

Copy link
Member

bradfitz commented Jan 23, 2019

we need to make a CL to add them to this list (so that they can run the release command) and redeploy golang.org

And cherry-pick it to the release branch, IIRC? /cc @dmitshur

Potential places to store this info:

  • GCE metadata
  • GCS text/json file
  • ... ?

I think GCE metadata is my vote.

@bradfitz

This comment has been minimized.

Copy link
Member

bradfitz commented Jan 23, 2019

Or we don't use the gomote-style user authentication and instead use OAuth2-authenticated requests and then check some IAM permission.

/cc @FiloSottile

@dmitshur

This comment has been minimized.

Copy link
Member

dmitshur commented Jan 23, 2019

And cherry-pick it to the release branch, IIRC? /cc @dmitshur

Yes, and then deploy golang.org. This becomes simpler after #29206 is done.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment