internal: don't set client_id and client_secret form values if empty

Fixes golang/oauth2#220.

Change-Id: Ic43b10971e102a8571c7bc895c3ad02b80b685ee
Reviewed-by: Chris Broadfoot <>
2 files changed
tree: 388ac6219d77e3b371726f30df35cd9d4532b690
  1. .travis.yml
  7. bitbucket/
  8. client_appengine.go
  9. clientcredentials/
  10. example_test.go
  11. facebook/
  12. fitbit/
  13. foursquare/
  14. github/
  15. google/
  16. heroku/
  17. hipchat/
  18. internal/
  19. jws/
  20. jwt/
  21. linkedin/
  22. mediamath/
  23. microsoft/
  24. oauth2.go
  25. oauth2_test.go
  26. odnoklassniki/
  27. paypal/
  28. slack/
  29. token.go
  30. token_test.go
  31. transport.go
  32. transport_test.go
  33. uber/
  34. vk/
  35. yandex/

OAuth2 for Go

Build Status GoDoc

oauth2 package contains a client implementation for OAuth 2.0 spec.


go get

See godoc for further documentation and examples.

App Engine

In change 96e89be (March 2015) we removed the oauth2.Context2 type in favor of the context.Context type from the package

This means its no longer possible to use the “Classic App Engine” appengine.Context type with the oauth2 package. (You're using Classic App Engine if you import the package "appengine".)

To work around this, you may use the new "" package. This package has almost the same API as the "appengine" package, but it can be fetched with go get and used on “Managed VMs” and well as Classic App Engine.

See the new appengine package's readme for information on updating your app.

If you don't want to update your entire app to use the new App Engine packages, you may use both sets of packages in parallel, using only the new packages with the oauth2 package.

import (
	newappengine ""
	newurlfetch ""


func handler(w http.ResponseWriter, r *http.Request) {
	var c appengine.Context = appengine.NewContext(r)
	c.Infof("Logging a message with the old package")

	var ctx context.Context = newappengine.NewContext(r)
	client := &http.Client{
		Transport: &oauth2.Transport{
			Source: google.AppEngineTokenSource(ctx, "scope"),
			Base:   &newurlfetch.Transport{Context: ctx},