Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(1825)

Side by Side Diff: appengine/gaemiddleware/routes.go

Issue 2576923003: Implement config service cache on top of datastore (Closed)
Patch Set: Relocated, fix, split integration test, rebase. Created 3 years, 11 months ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View unified diff | Download patch
« no previous file with comments | « appengine/gaemiddleware/resources.cfg ('k') | common/testing/assertions/error_tests.go » ('j') | no next file with comments »
Toggle Intra-line Diffs ('i') | Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
OLDNEW
1 // Copyright 2015 The LUCI Authors. All rights reserved. 1 // Copyright 2015 The LUCI Authors. All rights reserved.
2 // Use of this source code is governed under the Apache License, Version 2.0 2 // Use of this source code is governed under the Apache License, Version 2.0
3 // that can be found in the LICENSE file. 3 // that can be found in the LICENSE file.
4 4
5 package gaemiddleware 5 package gaemiddleware
6 6
7 import ( 7 import (
8 "github.com/luci/luci-go/server/router" 8 "github.com/luci/luci-go/server/router"
9 "github.com/luci/luci-go/server/settings/admin" 9 "github.com/luci/luci-go/server/settings/admin"
10 10
11 gaeauth "github.com/luci/luci-go/appengine/gaeauth/server" 11 gaeauth "github.com/luci/luci-go/appengine/gaeauth/server"
12 "github.com/luci/luci-go/appengine/tsmon" 12 "github.com/luci/luci-go/appengine/tsmon"
13 "github.com/luci/luci-go/luci_config/appengine/gaeconfig"
13 ) 14 )
14 15
15 // InstallHandlers installs HTTP handlers for various default routes. 16 // InstallHandlers installs HTTP handlers for various default routes.
16 // 17 //
17 // These routes are needed for various services provided in BaseProd context to 18 // These routes are needed for various services provided in BaseProd context to
18 // work (e.g. authentication related routes, time series monitoring, etc). 19 // work (e.g. authentication related routes, time series monitoring, etc).
19 // 20 //
20 // 'base' is expected to be BaseProd() or its derivative. It must NOT do any 21 // 'base' is expected to be BaseProd() or its derivative. It must NOT do any
21 // interception of requests (e.g. checking and rejecting unauthenticated 22 // interception of requests (e.g. checking and rejecting unauthenticated
22 // requests). It may inject additional state in the context though, if it is 23 // requests). It may inject additional state in the context though, if it is
23 // needed by various tsmon callbacks or settings pages. 24 // needed by various tsmon callbacks or settings pages.
24 // 25 //
25 // In majority of cases 'base' should just be BaseProd(). If unsure what to use, 26 // In majority of cases 'base' should just be BaseProd(). If unsure what to use,
26 // use BaseProd(). 27 // use BaseProd().
27 func InstallHandlers(r *router.Router, base router.MiddlewareChain) { 28 func InstallHandlers(r *router.Router, base router.MiddlewareChain) {
28 gaeauth.InstallHandlers(r, base) 29 gaeauth.InstallHandlers(r, base)
29 tsmon.InstallHandlers(r, base) 30 tsmon.InstallHandlers(r, base)
30 admin.InstallHandlers(r, base, &gaeauth.UsersAPIAuthMethod{}) 31 admin.InstallHandlers(r, base, &gaeauth.UsersAPIAuthMethod{})
32 gaeconfig.InstallCacheCronHandler(r, base.Extend(RequireCron))
31 } 33 }
OLDNEW
« no previous file with comments | « appengine/gaemiddleware/resources.cfg ('k') | common/testing/assertions/error_tests.go » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698