core.sr.ht: Allow id attribute on sanitized anchor tags v1 APPLIED

Nolan Prescott: 1
 Allow id attribute on sanitized anchor tags

 1 files changed, 2 insertions(+), 3 deletions(-)
#362894 alpine.yml success
#362895 archlinux.yml success
#362896 debian.yml success
Export patchset (mbox)
How do I use this?

Copy & paste the following snippet into your terminal to import this patchset into git:

curl -s https://lists.sr.ht/~sircmpwn/sr.ht-dev/patches/15867/mbox | git am -3
Learn more about email & git

[PATCH core.sr.ht] Allow id attribute on sanitized anchor tags Export this patch

Previously the sanitizer attributes dictionary was created by updating
the desired whitelist with `bleach.sanitizer.ALLOWED_ATTRIBUTES` which
is (at the time of this writing):

    {'a': ['href', 'title'], 'abbr': ['title'], 'acronym': ['title']}

By updating the whitelist dictionary with a duplicated key (`a`), the
whitelist value is overwritten.

Instead, build a new dictionary by merging the two dictionaries with a
preference for the whitelisted values. To accommodate the overwrite
behavior the whitelist is expanded to include those default values.
 srht/markdown.py | 5 ++---
 1 file changed, 2 insertions(+), 3 deletions(-)

diff --git a/srht/markdown.py b/srht/markdown.py
index a724e4f..df3d450 100644
--- a/srht/markdown.py
+++ b/srht/markdown.py
@@ -139,7 +139,7 @@ def _wildcard_filter(tag, name, value):
    return name in ["style", "class", "colspan", "rowspan"]

_sanitizer_attrs = {
    "a": ["id"],
    "a": ["id", "href", "title"],
    "h1": ["id"],
    "h2": ["id"],
    "h3": ["id"],
@@ -150,7 +150,6 @@ _sanitizer_attrs = {
    "input": _input_filter,
    "*": _wildcard_filter,
_sanitizer = bleach.sanitizer.Cleaner(
    tags=bleach.sanitizer.ALLOWED_TAGS + [
        "p", "div", "span", "pre", "hr",
@@ -161,7 +160,7 @@ _sanitizer = bleach.sanitizer.Cleaner(
        "h1", "h2", "h3", "h4", "h5", "h6",
    attributes={**bleach.sanitizer.ALLOWED_ATTRIBUTES, **_sanitizer_attrs},
core.sr.ht/patches: SUCCESS in 4m52s

[Allow id attribute on sanitized anchor tags][0] from [Nolan Prescott][1]

[0]: https://lists.sr.ht/~sircmpwn/sr.ht-dev/patches/15867
[1]: mailto:mail@nprescott.com

✓ #362894 SUCCESS core.sr.ht/patches/alpine.yml    https://builds.sr.ht/~sircmpwn/job/362894
✓ #362895 SUCCESS core.sr.ht/patches/archlinux.yml https://builds.sr.ht/~sircmpwn/job/362895
✓ #362896 SUCCESS core.sr.ht/patches/debian.yml    https://builds.sr.ht/~sircmpwn/job/362896

To git@git.sr.ht:~sircmpwn/core.sr.ht
   9a95abb..fe16d27  master -> master