[PR #2956] [MERGED] Fix over-escaped characters #16682

Closed
opened 2025-11-02 12:16:16 -06:00 by GiteaMirror · 0 comments
Owner

📋 Pull Request Information

Original PR: https://github.com/go-gitea/gitea/pull/2956
Author: @ethantkoenig
Created: 11/22/2017
Status: Merged
Merged: 11/22/2017
Merged by: @lunny

Base: masterHead: fix/html_escape


📝 Commits (2)

  • 6468693 Fix over-escaped characters
  • 168c2b0 Merge branch 'master' into fix/html_escape

📊 Changes

5 files changed (+17 additions, -19 deletions)

View changed files

📝 modules/templates/helper.go (+4 -6)
📝 templates/repo/issue/list.tmpl (+6 -6)
📝 templates/repo/issue/milestones.tmpl (+1 -1)
📝 templates/repo/issue/view_content/sidebar.tmpl (+4 -4)
📝 templates/repo/wiki/view.tmpl (+2 -2)

📄 Description

Fixes #2872. We do not need to call Sanitize in templates, because the html/template library already escapes strings for us.


🔄 This issue represents a GitHub Pull Request. It cannot be merged through Gitea due to API limitations.

## 📋 Pull Request Information **Original PR:** https://github.com/go-gitea/gitea/pull/2956 **Author:** [@ethantkoenig](https://github.com/ethantkoenig) **Created:** 11/22/2017 **Status:** ✅ Merged **Merged:** 11/22/2017 **Merged by:** [@lunny](https://github.com/lunny) **Base:** `master` ← **Head:** `fix/html_escape` --- ### 📝 Commits (2) - [`6468693`](https://github.com/go-gitea/gitea/commit/6468693b59fd1a21cc2cb4a0169b18a2a900b87a) Fix over-escaped characters - [`168c2b0`](https://github.com/go-gitea/gitea/commit/168c2b0b8a3356a67153ca13897a2ff28529b8fa) Merge branch 'master' into fix/html_escape ### 📊 Changes **5 files changed** (+17 additions, -19 deletions) <details> <summary>View changed files</summary> 📝 `modules/templates/helper.go` (+4 -6) 📝 `templates/repo/issue/list.tmpl` (+6 -6) 📝 `templates/repo/issue/milestones.tmpl` (+1 -1) 📝 `templates/repo/issue/view_content/sidebar.tmpl` (+4 -4) 📝 `templates/repo/wiki/view.tmpl` (+2 -2) </details> ### 📄 Description Fixes #2872. We do not need to call `Sanitize` in templates, because the [html/template](https://golang.org/pkg/html/template/) library already escapes strings for us. --- <sub>🔄 This issue represents a GitHub Pull Request. It cannot be merged through Gitea due to API limitations.</sub>
GiteaMirror added the pull-request label 2025-11-02 12:16:16 -06:00
Sign in to join this conversation.
1 Participants
Notifications
Due Date
No due date set.
Dependencies

No dependencies set.

Reference: github-starred/gitea#16682