Merge pull request #682 from reviewdog/share-round-trip

doghouse: Use shared http.RoundTripper

doghouse/appengine/checker.go

@@ -9,19 +9,18 @@ import (
 	"net/url"
 	"strings"
 
-	"contrib.go.opencensus.io/exporter/stackdriver/propagation"
 	"github.com/reviewdog/reviewdog/doghouse"
 	"github.com/reviewdog/reviewdog/doghouse/server"
 	"github.com/reviewdog/reviewdog/doghouse/server/ciutil"
 	"github.com/reviewdog/reviewdog/doghouse/server/storage"
-	"go.opencensus.io/plugin/ochttp"
 )
 
 type githubChecker struct {
 	privateKey       []byte
 	integrationID    int
 	ghInstStore      storage.GitHubInstallationStore
 	ghRepoTokenStore storage.GitHubRepositoryTokenStore
+	tr               http.RoundTripper
 }
 
 func (gc *githubChecker) handleCheck(w http.ResponseWriter, r *http.Request) {
@@ -48,10 +47,7 @@ func (gc *githubChecker) handleCheck(w http.ResponseWriter, r *http.Request) {
 		IntegrationID: gc.integrationID,
 		RepoOwner:     req.Owner,
 		Client: &http.Client{
-			Transport: &ochttp.Transport{
-				// Use Google Cloud propagation format.
-				Propagation: &propagation.HTTPFormat{},
-			},
+			Transport: gc.tr,
 		},
 	}
 

doghouse/appengine/main.go

@@ -86,6 +86,10 @@ func main() {
 		integrationID:    integrationID,
 		ghInstStore:      &ghInstStore,
 		ghRepoTokenStore: &ghRepoTokenStore,
+		tr: &ochttp.Transport{
+			// Use Google Cloud propagation format.
+			Propagation: &propagation.HTTPFormat{},
+		},
 	}
 
 	ghWebhookHandler := githubWebhookHandler{