|
|
|
goauth2: support client_credentials grant_type
Patch Set 1 #Patch Set 2 : diff -r 696c08849124 https://code.google.com/p/goauth2 #Patch Set 3 : diff -r 696c08849124 https://code.google.com/p/goauth2 #
Total comments: 2
Patch Set 4 : diff -r 696c08849124 https://code.google.com/p/goauth2 #Total messages: 12
|
mmlb
|
11 years, 7 months ago (2014年06月11日 13:44:09 UTC) #1 |
Hello golang-dev@googlegroups.com, I'd like you to review this change to https://code.google.com/p/goauth2
Replacing golang-dev with golang-codereviews. To the author of this CL: If you are using 'hg mail -r golang-dev' to mail the CL, use simply 'hg mail' instead. If you did not name golang-dev explicitly and it was still added to the CL, it means your working copy of the repo has a stale codereview.cfg (or lib/codereview/codereview.cfg). Please run 'hg sync' to update your client to the most recent codereview.cfg. If the most recent codereview.cfg has defaultcc set to golang-dev instead of golang-codereviews, please send a CL correcting it. Thanks very much.
R=adg
Related to https://github.com/golang/oauth2/issues/7.
https://codereview.appspot.com/103240043/diff/40001/oauth/oauth.go File oauth/oauth.go (right): https://codereview.appspot.com/103240043/diff/40001/oauth/oauth.go#newcode320 oauth/oauth.go:320: //AuthenticateClient gets an access Token using the client_credentials grant add a space after each // https://codereview.appspot.com/103240043/diff/40001/oauth/oauth.go#newcode329 oauth/oauth.go:329: return t.updateToken(t.Token, unwrap this; Go code may be longer than 80 cols or if you don't like that v := url.Values{...} return t.updateToken(t.Token, v)
Hello golang-codereviews@googlegroups.com, adg@golang.org, jbd@google.com (cc: golang-dev@googlegroups.com), Please take another look.
Replacing golang-dev with golang-codereviews. To the author of this CL: If you are using 'hg mail -r golang-dev' to mail the CL, use simply 'hg mail' instead. If you did not name golang-dev explicitly and it was still added to the CL, it means your working copy of the repo has a stale codereview.cfg (or lib/codereview/codereview.cfg). Please run 'hg sync' to update your client to the most recent codereview.cfg. If the most recent codereview.cfg has defaultcc set to golang-dev instead of golang-codereviews, please send a CL correcting it. Thanks very much.
Hey MMendez534 - have you signed the CLA? http://golang.org/doc/contribute.html#copyright
Yup I've signed it. On Wed, Jul 9, 2014 at 12:07 AM, Andrew Gerrand <adg@golang.org> wrote: > Hey MMendez534 - have you signed the CLA? > > http://golang.org/doc/contribute.html#copyright >
LGTM
*** Submitted as https://code.google.com/p/goauth2/source/detail?r=6a3615e294b5 *** goauth2: support client_credentials grant_type LGTM=adg R=golang-codereviews, adg, jbd CC=golang-codereviews https://codereview.appspot.com/103240043 Committer: Andrew Gerrand <adg@golang.org>