X-Git-Url: https://git.arvados.org/arvados.git/blobdiff_plain/3f32ceb98c74d4c1ad056615c3dac359ad0c0bce..f690cb6af6d36a5105a846dd69e0fd624b4e7f0e:/lib/controller/handler.go diff --git a/lib/controller/handler.go b/lib/controller/handler.go index 358b0ed0c2..665fd5c636 100644 --- a/lib/controller/handler.go +++ b/lib/controller/handler.go @@ -13,7 +13,6 @@ import ( "net/url" "strings" "sync" - "time" "git.arvados.org/arvados.git/lib/controller/api" "git.arvados.org/arvados.git/lib/controller/federation" @@ -32,9 +31,11 @@ import ( ) type Handler struct { - Cluster *arvados.Cluster + Cluster *arvados.Cluster + BackgroundContext context.Context setupOnce sync.Once + federation *federation.Conn handlerStack http.Handler proxy *proxy secureClient *http.Client @@ -59,12 +60,6 @@ func (h *Handler) ServeHTTP(w http.ResponseWriter, req *http.Request) { req.URL.Path = strings.Replace(req.URL.Path, "//", "/", -1) } } - if h.Cluster.API.RequestTimeout > 0 { - ctx, cancel := context.WithDeadline(req.Context(), time.Now().Add(time.Duration(h.Cluster.API.RequestTimeout))) - req = req.WithContext(ctx) - defer cancel() - } - h.handlerStack.ServeHTTP(w, req) } @@ -100,17 +95,24 @@ func neverRedirect(*http.Request, []*http.Request) error { return http.ErrUseLas func (h *Handler) setup() { mux := http.NewServeMux() - mux.Handle("/_health/", &health.Handler{ - Token: h.Cluster.ManagementToken, - Prefix: "/_health/", - Routes: health.Routes{"ping": func() error { _, err := h.db(context.TODO()); return err }}, - }) + healthFuncs := make(map[string]health.Func) oidcAuthorizer := localdb.OIDCAccessTokenAuthorizer(h.Cluster, h.db) - rtr := router.New(federation.New(h.Cluster), router.Config{ + h.federation = federation.New(h.Cluster, &healthFuncs) + rtr := router.New(h.federation, router.Config{ MaxRequestSize: h.Cluster.API.MaxRequestSize, WrapCalls: api.ComposeWrappers(ctrlctx.WrapCallsInTransactions(h.db), oidcAuthorizer.WrapCalls), }) + + healthRoutes := health.Routes{"ping": func() error { _, err := h.db(context.TODO()); return err }} + for name, f := range healthFuncs { + healthRoutes[name] = f + } + mux.Handle("/_health/", &health.Handler{ + Token: h.Cluster.ManagementToken, + Prefix: "/_health/", + Routes: healthRoutes, + }) mux.Handle("/arvados/v1/config", rtr) mux.Handle("/arvados/v1/vocabulary", rtr) mux.Handle("/"+arvados.EndpointUserAuthenticate.Path, rtr) // must come before .../users/ @@ -127,6 +129,8 @@ func (h *Handler) setup() { mux.Handle("/arvados/v1/links/", rtr) mux.Handle("/login", rtr) mux.Handle("/logout", rtr) + mux.Handle("/arvados/v1/api_client_authorizations", rtr) + mux.Handle("/arvados/v1/api_client_authorizations/", rtr) hs := http.NotFoundHandler() hs = prepend(hs, h.proxyRailsAPI) @@ -146,6 +150,8 @@ func (h *Handler) setup() { h.proxy = &proxy{ Name: "arvados-controller", } + + go h.trashSweepWorker() } var errDBConnection = errors.New("database connection error")