From ad7fbaae7a2001502582e6daa27eaffad8a3c447 Mon Sep 17 00:00:00 2001 From: tjpcc Date: Wed, 18 Oct 2023 12:49:14 -0600 Subject: fix Modifiers.Empty() and "git with templates" --- parse.go | 13 +++++++++++-- types.go | 10 +++++++++- 2 files changed, 20 insertions(+), 3 deletions(-) diff --git a/parse.go b/parse.go index 886aa2d..e1b375f 100644 --- a/parse.go +++ b/parse.go @@ -265,9 +265,18 @@ func parseServerDirectives(server *Server, buf *bufio.Reader) error { return nil } +func emptyExceptTemplates(mods *Modifiers) bool { + if mods == nil { + return true + } + cpy := *mods + cpy.Templates = nil + return cpy.Empty() +} + func validateRoute(serverType string, dir *RouteDirective) error { - if dir.Type == "git" && !dir.Modifiers.Empty() { - return errors.New("git directives don't support 'with' modifiers") + if dir.Type == "git" && !emptyExceptTemplates(&dir.Modifiers) { + return errors.New("unsupported 'with' modifier on 'git' directive") } if dir.Type == "cgi" && (dir.Modifiers.Exec || dir.Modifiers.DirList || dir.Modifiers.DirDefault != "") { diff --git a/types.go b/types.go index 392b955..af7f30f 100644 --- a/types.go +++ b/types.go @@ -25,7 +25,15 @@ type Modifiers struct { } func (m Modifiers) Empty() bool { - return m.DirDefault == "" && !m.DirList && !m.Exec && !m.ExtendedGophermap + return (m.DirDefault == "" && + !m.DirList && + !m.Exec && + !m.ExtendedGophermap && + m.ExecCmd == "" && + !m.AutoAtom && + m.Titan == nil && + m.titanName == "" && + m.Templates == nil) } type RouteDirective struct { -- cgit v1.2.3