diff options
Diffstat (limited to 'api4/context.go')
-rw-r--r-- | api4/context.go | 18 |
1 files changed, 16 insertions, 2 deletions
diff --git a/api4/context.go b/api4/context.go index 19778dda3..62fe55758 100644 --- a/api4/context.go +++ b/api4/context.go @@ -212,8 +212,10 @@ func (c *Context) LogAuditWithUserId(userId, extraInfo string) { func (c *Context) LogError(err *model.AppError) { - // filter out endless reconnects - if c.Path == "/api/v3/users/websocket" && err.StatusCode == 401 || err.Id == "web.check_browser_compatibility.app_error" { + // Filter out 404s, endless reconnects and browser compatibility errors + if err.StatusCode == http.StatusNotFound || + (c.Path == "/api/v3/users/websocket" && err.StatusCode == 401) || + err.Id == "web.check_browser_compatibility.app_error" { c.LogDebug(err) } else { l4g.Error(utils.TDefault("api.context.log.error"), c.Path, err.Where, err.StatusCode, @@ -447,6 +449,18 @@ func (c *Context) RequireFileId() *Context { return c } +func (c *Context) RequireFilename() *Context { + if c.Err != nil { + return c + } + + if len(c.Params.Filename) == 0 { + c.SetInvalidUrlParam("filename") + } + + return c +} + func (c *Context) RequirePluginId() *Context { if c.Err != nil { return c |