From c88392e7729f30ee55c4be4885bc2324f9fd009c Mon Sep 17 00:00:00 2001 From: zeripath Date: Wed, 12 May 2021 20:54:50 +0100 Subject: [PATCH] Upgrade unrolled/render to v1.1.1 (#15845) (#15846) Backport #15845 Co-authored-by: Lunny Xiao --- go.mod | 2 +- go.sum | 6 ++--- vendor/github.com/unrolled/render/render.go | 27 ++++++++++++++------- vendor/modules.txt | 2 +- 4 files changed, 22 insertions(+), 15 deletions(-) diff --git a/go.mod b/go.mod index f03ca60d9d..8be16a02e0 100644 --- a/go.mod +++ b/go.mod @@ -122,7 +122,7 @@ require ( github.com/unknwon/com v1.0.1 github.com/unknwon/i18n v0.0.0-20200823051745-09abd91c7f2c github.com/unknwon/paginater v0.0.0-20200328080006-042474bd0eae - github.com/unrolled/render v1.1.0 + github.com/unrolled/render v1.1.1 github.com/urfave/cli v1.22.5 github.com/willf/bitset v1.1.11 // indirect github.com/xanzy/go-gitlab v0.44.0 diff --git a/go.sum b/go.sum index 46200815e8..ebf4ced2b7 100644 --- a/go.sum +++ b/go.sum @@ -1113,10 +1113,8 @@ github.com/unknwon/i18n v0.0.0-20200823051745-09abd91c7f2c h1:679/gJXwrsHC3RATr0 github.com/unknwon/i18n v0.0.0-20200823051745-09abd91c7f2c/go.mod h1:+5rDk6sDGpl3azws3O+f+GpFSyN9GVr0K8cvQLQM2ZQ= github.com/unknwon/paginater v0.0.0-20200328080006-042474bd0eae h1:ihaXiJkaca54IaCSnEXtE/uSZOmPxKZhDfVLrzZLFDs= github.com/unknwon/paginater v0.0.0-20200328080006-042474bd0eae/go.mod h1:1fdkY6xxl6ExVs2QFv7R0F5IRZHKA8RahhB9fMC9RvM= -github.com/unrolled/render v1.0.3 h1:baO+NG1bZSF2WR4zwh+0bMWauWky7DVrTOfvE2w+aFo= -github.com/unrolled/render v1.0.3/go.mod h1:gN9T0NhL4Bfbwu8ann7Ry/TGHYfosul+J0obPf6NBdM= -github.com/unrolled/render v1.1.0 h1:gvpR9hHxTt6DcGqRYuVVFcfd8rtK+nyEPUJN06KB57Q= -github.com/unrolled/render v1.1.0/go.mod h1:gN9T0NhL4Bfbwu8ann7Ry/TGHYfosul+J0obPf6NBdM= +github.com/unrolled/render v1.1.1 h1:FpzNzkvlJQIlVdVaqeVBGWiCS8gpbmjtrKpDmCn6p64= +github.com/unrolled/render v1.1.1/go.mod h1:gN9T0NhL4Bfbwu8ann7Ry/TGHYfosul+J0obPf6NBdM= github.com/urfave/cli v1.20.0/go.mod h1:70zkFmudgCuE/ngEzBv17Jvp/497gISqfk5gWijbERA= github.com/urfave/cli v1.22.1/go.mod h1:Gos4lmkARVdJ6EkW0WaNv/tZAAMe9V7XWyB60NtXRu0= github.com/urfave/cli v1.22.5 h1:lNq9sAHXK2qfdI8W+GRItjCEkI+2oR4d+MEHy1CKXoU= diff --git a/vendor/github.com/unrolled/render/render.go b/vendor/github.com/unrolled/render/render.go index 4cd11cdfcc..ad2e77ee19 100644 --- a/vendor/github.com/unrolled/render/render.go +++ b/vendor/github.com/unrolled/render/render.go @@ -123,7 +123,7 @@ type Render struct { // Customize Secure with an Options struct. opt Options templates *template.Template - templatesLk sync.Mutex + templatesLk sync.RWMutex compiledCharset string } @@ -196,8 +196,8 @@ func (r *Render) compileTemplates() { func (r *Render) compileTemplatesFromDir() { dir := r.opt.Directory - r.templates = template.New(dir) - r.templates.Delims(r.opt.Delims.Left, r.opt.Delims.Right) + tmpTemplates := template.New(dir) + tmpTemplates.Delims(r.opt.Delims.Left, r.opt.Delims.Right) // Walk the supplied directory and compile any files that match our extension list. r.opt.FileSystem.Walk(dir, func(path string, info os.FileInfo, err error) error { @@ -227,7 +227,7 @@ func (r *Render) compileTemplatesFromDir() { } name := (rel[0 : len(rel)-len(ext)]) - tmpl := r.templates.New(filepath.ToSlash(name)) + tmpl := tmpTemplates.New(filepath.ToSlash(name)) // Add our funcmaps. for _, funcs := range r.opt.Funcs { @@ -241,12 +241,16 @@ func (r *Render) compileTemplatesFromDir() { } return nil }) + + r.templatesLk.Lock() + r.templates = tmpTemplates + r.templatesLk.Unlock() } func (r *Render) compileTemplatesFromAsset() { dir := r.opt.Directory - r.templates = template.New(dir) - r.templates.Delims(r.opt.Delims.Left, r.opt.Delims.Right) + tmpTemplates := template.New(dir) + tmpTemplates.Delims(r.opt.Delims.Left, r.opt.Delims.Right) for _, path := range r.opt.AssetNames() { if !strings.HasPrefix(path, dir) { @@ -272,7 +276,7 @@ func (r *Render) compileTemplatesFromAsset() { } name := (rel[0 : len(rel)-len(ext)]) - tmpl := r.templates.New(filepath.ToSlash(name)) + tmpl := tmpTemplates.New(filepath.ToSlash(name)) // Add our funcmaps. for _, funcs := range r.opt.Funcs { @@ -285,6 +289,10 @@ func (r *Render) compileTemplatesFromAsset() { } } } + + r.templatesLk.Lock() + r.templates = tmpTemplates + r.templatesLk.Unlock() } // TemplateLookup is a wrapper around template.Lookup and returns @@ -389,14 +397,15 @@ func (r *Render) Data(w io.Writer, status int, v []byte) error { // HTML builds up the response from the specified template and bindings. func (r *Render) HTML(w io.Writer, status int, name string, binding interface{}, htmlOpt ...HTMLOptions) error { - r.templatesLk.Lock() - defer r.templatesLk.Unlock() // If we are in development mode, recompile the templates on every HTML request. if r.opt.IsDevelopment { r.compileTemplates() } + r.templatesLk.RLock() + defer r.templatesLk.RUnlock() + opt := r.prepareHTMLOptions(htmlOpt) if tpl := r.templates.Lookup(name); tpl != nil { if len(opt.Layout) > 0 { diff --git a/vendor/modules.txt b/vendor/modules.txt index f5ea4af54a..d9eca1e25a 100644 --- a/vendor/modules.txt +++ b/vendor/modules.txt @@ -777,7 +777,7 @@ github.com/unknwon/i18n # github.com/unknwon/paginater v0.0.0-20200328080006-042474bd0eae ## explicit github.com/unknwon/paginater -# github.com/unrolled/render v1.1.0 +# github.com/unrolled/render v1.1.1 ## explicit github.com/unrolled/render # github.com/urfave/cli v1.22.5