Revert "Use in-memory http caching" (#127)

The current cache implementation has some issues with both authenticated requests and freshness of data, so reverting it while we work on improvements. This reverts commit 5bebbba.
palantir · Jun 13, 2019 · 2c872f6 · 2c872f6
1 parent 5bebbba
commit 2c872f6
Show file tree

Hide file tree

Showing 6 changed files with 2 additions and 590 deletions.
diff --git a/.gitignore b/.gitignore
@@ -3,6 +3,5 @@
 *.iml
 *.ipr
 *.iws
-config/bulldozer.yml
 
 build/
diff --git a/Gopkg.lock b/Gopkg.lock
diff --git a/server/server.go b/server/server.go
@@ -17,7 +17,6 @@ package server
 import (
 	"fmt"
 
-	"github.com/gregjones/httpcache"
 	"github.com/palantir/go-baseapp/baseapp"
 	"github.com/palantir/go-baseapp/baseapp/datadog"
 	"github.com/palantir/go-githubapp/githubapp"
@@ -54,7 +53,6 @@ func New(c *Config) (*Server, error) {
 		c.Github,
 		githubapp.WithClientUserAgent(userAgent),
 		githubapp.WithClientMiddleware(
-			githubapp.ClientCaching(func() httpcache.Cache { return httpcache.NewMemoryCache() }),
 			githubapp.ClientLogging(zerolog.DebugLevel),
 			githubapp.ClientMetrics(base.Registry()),
 		),

diff --git a/vendor/github.com/gregjones/httpcache/LICENSE.txt b/vendor/github.com/gregjones/httpcache/LICENSE.txt
-Original file line number
+Diff line change
@@ Expand Up / @@ -3,6 +3,5 @@ @@
     *.iml
     *.ipr
     *.iws
-    config/bulldozer.yml
     build/