diff --git a/pkg/api/dashboard_snapshot.go b/pkg/api/dashboard_snapshot.go index 434e85a46c4..09e441fb63b 100644 --- a/pkg/api/dashboard_snapshot.go +++ b/pkg/api/dashboard_snapshot.go @@ -147,6 +147,10 @@ func CreateDashboardSnapshot(c *models.ReqContext, cmd models.CreateDashboardSna // GET /api/snapshots/:key func GetDashboardSnapshot(c *models.ReqContext) response.Response { key := macaron.Params(c.Req)[":key"] + if len(key) == 0 { + return response.Error(404, "Snapshot not found", nil) + } + query := &models.GetDashboardSnapshotQuery{Key: key} err := bus.Dispatch(query) @@ -211,6 +215,9 @@ func deleteExternalDashboardSnapshot(externalUrl string) error { // GET /api/snapshots-delete/:deleteKey func DeleteDashboardSnapshotByDeleteKey(c *models.ReqContext) response.Response { key := macaron.Params(c.Req)[":deleteKey"] + if len(key) == 0 { + return response.Error(404, "Snapshot not found", nil) + } query := &models.GetDashboardSnapshotQuery{DeleteKey: key} @@ -241,6 +248,9 @@ func DeleteDashboardSnapshotByDeleteKey(c *models.ReqContext) response.Response // DELETE /api/snapshots/:key func DeleteDashboardSnapshot(c *models.ReqContext) response.Response { key := macaron.Params(c.Req)[":key"] + if len(key) == 0 { + return response.Error(404, "Snapshot not found", nil) + } query := &models.GetDashboardSnapshotQuery{Key: key} diff --git a/pkg/macaron/router.go b/pkg/macaron/router.go index 778bb99ff87..18a87ce9ea7 100644 --- a/pkg/macaron/router.go +++ b/pkg/macaron/router.go @@ -207,10 +207,12 @@ func (r *Router) NotFound(handlers ...Handler) { func (r *Router) ServeHTTP(rw http.ResponseWriter, req *http.Request) { if t, ok := r.routers[req.Method]; ok { // Fast match for static routes - leaf := r.getLeaf(req.Method, req.URL.Path) - if leaf != nil { - leaf.handle(rw, req, nil) - return + if !strings.ContainsAny(req.URL.Path, ":*") { + leaf := r.getLeaf(req.Method, req.URL.Path) + if leaf != nil { + leaf.handle(rw, req, nil) + return + } } h, p, ok := t.Match(req.URL.EscapedPath())