Skip to content

Commit

Permalink
Exclude event attributes when rendering markdown
Browse files Browse the repository at this point in the history
Closes #9463
  • Loading branch information
jmooring authored and bep committed Feb 10, 2022
1 parent 54f8d8a commit f7bc4cc
Show file tree
Hide file tree
Showing 3 changed files with 64 additions and 1 deletion.
1 change: 1 addition & 0 deletions .gitignore
Original file line number Diff line number Diff line change
Expand Up @@ -26,3 +26,4 @@ resources/sunset.jpg

vendor

.hugo_build.lock
61 changes: 61 additions & 0 deletions markup/goldmark/integration_test.go
Original file line number Diff line number Diff line change
@@ -0,0 +1,61 @@
// Copyright 2021 The Hugo Authors. All rights reserved.
//
// Licensed under the Apache License, Version 2.0 (the "License");
// you may not use this file except in compliance with the License.
// You may obtain a copy of the License at
// http://www.apache.org/licenses/LICENSE-2.0
//
// Unless required by applicable law or agreed to in writing, software
// distributed under the License is distributed on an "AS IS" BASIS,
// WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied.
// See the License for the specific language governing permissions and
// limitations under the License.

package goldmark_test

import (
"testing"

"github.com/gohugoio/hugo/hugolib"
)

func TestAttributeExclusion(t *testing.T) {
t.Parallel()

files := `
-- config.toml --
[markup.goldmark.renderer]
unsafe = false
[markup.goldmark.parser.attribute]
block = true
title = true
-- content/p1.md --
---
title: "p1"
---
## Heading {class="a" onclick="alert('heading')" linenos="inline"}
> Blockquote
{class="b" ondblclick="alert('blockquote')" LINENOS="inline"}
~~~bash {id="c" onmouseover="alert('code fence')"}
foo
~~~
-- layouts/_default/single.html --
{{ .Content }}
`

b := hugolib.NewIntegrationTestBuilder(
hugolib.IntegrationTestConfig{
T: t,
TxtarString: files,
NeedsOsFS: false,
},
).Build()

b.AssertFileContent("public/p1/index.html", `
<h2 class="a" id="heading">
<blockquote class="b">
<div class="highlight" id="c">
`)
}
3 changes: 2 additions & 1 deletion markup/goldmark/render_hooks.go
Original file line number Diff line number Diff line change
Expand Up @@ -161,7 +161,8 @@ func renderAttributes(w util.BufWriter, skipClass bool, attributes ...ast.Attrib
continue
}

if attributeExcludes[string(attr.Name)] {
a := strings.ToLower(string(attr.Name))
if attributeExcludes[a] || strings.HasPrefix(a, "on") {
continue
}

Expand Down

0 comments on commit f7bc4cc

Please sign in to comment.