htwkmooc/poseidon
1
0
Fork 0
Code Issues Pull Requests Actions Packages Projects Releases Wiki Activity

Fix Bug that only runner routes get registered with authorization.

Browse Source
This commit is contained in:
Maximilian Paß
2021-10-11 20:44:19 +02:00
parent 791f997846
commit af506dd26f
1 changed files with 5 additions and 1 deletions
Download Patch File Download Diff File Expand all files Collapse all files

6
internal/api/api.go
View File

@ -34,6 +34,10 @@ func NewRouter(runnerManager runner.Manager, environmentManager environment.Mana
// configureV1Router configures a given router with the routes of version 1 of the Poseidon API. // configureV1Router configures a given router with the routes of version 1 of the Poseidon API.
func configureV1Router(router *mux.Router, runnerManager runner.Manager, environmentManager environment.Manager) { func configureV1Router(router *mux.Router, runnerManager runner.Manager, environmentManager environment.Manager) {
router.NotFoundHandler = http.HandlerFunc(func(w http.ResponseWriter, r *http.Request) {
log.WithField("request", r).Debug("Not Found Handler")
w.WriteHeader(http.StatusNotFound)
})
v1 := router.PathPrefix(BasePath).Subrouter() v1 := router.PathPrefix(BasePath).Subrouter()
v1.HandleFunc(HealthPath, Health).Methods(http.MethodGet) v1.HandleFunc(HealthPath, Health).Methods(http.MethodGet)
@ -50,7 +54,7 @@ func configureV1Router(router *mux.Router, runnerManager runner.Manager, environ
// All routes added to v1 after this require authentication. // All routes added to v1 after this require authentication.
authenticatedV1Router := v1.PathPrefix("").Subrouter() authenticatedV1Router := v1.PathPrefix("").Subrouter()
authenticatedV1Router.Use(auth.HTTPAuthenticationMiddleware) authenticatedV1Router.Use(auth.HTTPAuthenticationMiddleware)
runnerController.ConfigureRoutes(authenticatedV1Router) configureRoutes(authenticatedV1Router)
} else { } else {
configureRoutes(v1) configureRoutes(v1)
} }