X-Git-Url: https://git.arvados.org/arvados.git/blobdiff_plain/f30c8ed35e3e1ad7cb3cb51fc6d83f56a04ae8de..36b4a526be01a76eeb2ef2db645598f45f1de992:/lib/controller/handler.go diff --git a/lib/controller/handler.go b/lib/controller/handler.go index 5f6fb192e1..22d2e8329a 100644 --- a/lib/controller/handler.go +++ b/lib/controller/handler.go @@ -9,6 +9,7 @@ import ( "errors" "fmt" "net/http" + "net/http/httptest" "net/url" "strings" "sync" @@ -74,7 +75,21 @@ func (h *Handler) CheckHealth() error { return err } _, _, err = railsproxy.FindRailsAPI(h.Cluster) - return err + if err != nil { + return err + } + if h.Cluster.API.VocabularyPath != "" { + req, err := http.NewRequest("GET", "/arvados/v1/vocabulary", nil) + if err != nil { + return err + } + var resp httptest.ResponseRecorder + h.handlerStack.ServeHTTP(&resp, req) + if resp.Result().StatusCode != http.StatusOK { + return fmt.Errorf("%d %s", resp.Result().StatusCode, resp.Result().Status) + } + } + return nil } func (h *Handler) Done() <-chan struct{} { @@ -85,28 +100,38 @@ func neverRedirect(*http.Request, []*http.Request) error { return http.ErrUseLas func (h *Handler) setup() { mux := http.NewServeMux() + var vocHealthFunc health.Func + + oidcAuthorizer := localdb.OIDCAccessTokenAuthorizer(h.Cluster, h.db) + rtr := router.New(federation.New(h.Cluster, &vocHealthFunc), router.Config{ + MaxRequestSize: h.Cluster.API.MaxRequestSize, + WrapCalls: api.ComposeWrappers(ctrlctx.WrapCallsInTransactions(h.db), oidcAuthorizer.WrapCalls), + }) + mux.Handle("/_health/", &health.Handler{ Token: h.Cluster.ManagementToken, Prefix: "/_health/", - Routes: health.Routes{"ping": func() error { _, err := h.db(context.TODO()); return err }}, + Routes: health.Routes{ + "ping": func() error { _, err := h.db(context.TODO()); return err }, + "vocabulary": vocHealthFunc, + }, }) - - oidcAuthorizer := localdb.OIDCAccessTokenAuthorizer(h.Cluster, h.db) - rtr := router.New(federation.New(h.Cluster), api.ComposeWrappers(ctrlctx.WrapCallsInTransactions(h.db), oidcAuthorizer.WrapCalls)) mux.Handle("/arvados/v1/config", rtr) - mux.Handle("/"+arvados.EndpointUserAuthenticate.Path, rtr) - - if !h.Cluster.ForceLegacyAPI14 { - mux.Handle("/arvados/v1/collections", rtr) - mux.Handle("/arvados/v1/collections/", rtr) - mux.Handle("/arvados/v1/users", rtr) - mux.Handle("/arvados/v1/users/", rtr) - mux.Handle("/arvados/v1/connect/", rtr) - mux.Handle("/arvados/v1/container_requests", rtr) - mux.Handle("/arvados/v1/container_requests/", rtr) - mux.Handle("/login", rtr) - mux.Handle("/logout", rtr) - } + mux.Handle("/arvados/v1/vocabulary", rtr) + mux.Handle("/"+arvados.EndpointUserAuthenticate.Path, rtr) // must come before .../users/ + mux.Handle("/arvados/v1/collections", rtr) + mux.Handle("/arvados/v1/collections/", rtr) + mux.Handle("/arvados/v1/users", rtr) + mux.Handle("/arvados/v1/users/", rtr) + mux.Handle("/arvados/v1/connect/", rtr) + mux.Handle("/arvados/v1/container_requests", rtr) + mux.Handle("/arvados/v1/container_requests/", rtr) + mux.Handle("/arvados/v1/groups", rtr) + mux.Handle("/arvados/v1/groups/", rtr) + mux.Handle("/arvados/v1/links", rtr) + mux.Handle("/arvados/v1/links/", rtr) + mux.Handle("/login", rtr) + mux.Handle("/logout", rtr) hs := http.NotFoundHandler() hs = prepend(hs, h.proxyRailsAPI)