3176824910

Build Type

github

Committed by GitHub

Commit Message

<a href="https://github.com/tjcsl/ion/commit/<a class=hub.com/tjcsl/ion/commit/<a class="double-link" href="https://git"><a class=hub.com/tjcsl/ion/commit/<a class="double-link" href="https://git"><a class=hub.com/tjcsl/ion/commit/<a class="double-link" href="https://git"><a class=hub.com/tjcsl/ion/commit/a7fc519ff2e369778d6dce2a1f895c476dc6a468">a7fc519ff">&lt;a href=&quot;https://github.com/tjcsl/ion/commit/</a><a class="double-link" href="https://github.com/tjcsl/ion/commit/&lt;a class=&quot;double-link&quot; href=&quot;https://git">&lt;a class=</a>hub.com/tjcsl/ion/commit/&lt;a class=&quot;double-link&quot; href=&quot;https://git">&lt;a class=</a>hub.com/tjcsl/ion/commit/&lt;a class=&quot;double-link&quot; href=&quot;https://git">&lt;a class=</a>hub.com/tjcsl/ion/commit/a7fc519ff2e369778d6dce2a1f895c476dc6a468">a7fc519ff</a><a href="https://github.com/tjcsl/ion/commit/a7fc519ff2e369778d6dce2a1f895c476dc6a468">&quot;&gt;&amp;lt;a href=&amp;quot;https://github.com/tjcsl/ion/commit/&lt;/a&gt;&lt;a class=&quot;double-link&quot; href=&quot;https://github.com/tjcsl/ion/commit/&amp;lt;a class=&amp;quot;double-link&amp;quot; href=&amp;quot;https://git&quot;&gt;&amp;lt;a class=&lt;/a&gt;hub.com/tjcsl/ion/commit/&amp;lt;a class=&amp;quot;double-link&amp;quot; href=&amp;quot;https://git&quot;&gt;&amp;lt;a class=&lt;/a&gt;hub.com/tjcsl/ion/commit/&amp;lt;a class=&amp;quot;double-link&amp;quot; href=&amp;quot;https://git&quot;&gt;&amp;lt;a class=&lt;/a&gt;hub.com/tjcsl/ion/commit/&amp;lt;a class=&amp;quot;double-link&amp;quot; href=&amp;quot;https://git&quot;&gt;&amp;lt;a class=&lt;/a&gt;hub.com/tjcsl/ion/commit/&amp;lt;a class=&amp;quot;double-link&amp;quot; href=&amp;quot;https://git&quot;&gt;&amp;lt;a class=&lt;/a&gt;hub.com/tjcsl/ion/commit/a7fc519ff2e369778d6dce2a1f895c476dc6a468&quot;&gt;a7fc519ff&lt;/a&gt;&lt;a href=&quot;https://github.com/tjcsl/ion/commit/a7fc519ff2e369778d6dce2a1f895c476dc6a468&quot;&gt;&amp;quot;&amp;gt;&amp;amp;lt;a href=&amp;amp;quot;https://github.com/tjcsl/ion/commit/&amp;lt;/a&amp;gt;&amp;lt;a class=&amp;quot;double-link&amp;quot; href=&amp;quot;https://github.com/tjcsl/ion/commit/&amp;amp;lt;a class=&amp;amp;quot;double-link&amp;amp;quot; href=&amp;amp;quot;https://git&amp;quot;&amp;gt;&amp;amp... (continued)

Pull Request Pull Request #1416: Dev/formatting fixes & features

Run Details

2776 of 4631 branches covered (59.94%)

14 of 15 new or added lines in 5 files covered. (93.33%)

15326 of 18648 relevant lines covered (82.19%)

0.82 hits per line

Source File
Press 'n' to go to next uncovered line, 'b' for previous

38.71

/intranet/utils/html.py

import urllib.parse
from typing import Mapping, Optional, Tuple, Union

import bleach
from bleach.css_sanitizer import CSSSanitizer

tags = ["a", "abbr", "acronym", "b", "br", "blockquote", "code", "em", "hr", "i", "li", "ol", "strong", "ul", "iframe", "img", "div", "p"]

att = {
    "acronym": ["title"],
    "a": ["href", "title"],
    "abbr": ["title"],
    "iframe": ["src", "height", "width", "allowfullscreen", "frameborder"],
    "img": ["src", "alt", "title", "style"],
}

css_sanitizer = CSSSanitizer(
    allowed_css_properties=[
        "width",
        "height",
        "float",
        "margin-left",
        "margin-right",
        "margin-top",
        "margin-bottom",
        "margin",
        "border",
        "border-style",
        "border-width",
    ]
)


def safe_html(txt):
    return bleach.linkify(bleach.clean(txt, tags=tags, attributes=att, css_sanitizer=css_sanitizer))


def link_removal_callback(  # pylint: disable=unused-argument
    attrs: Mapping[Union[str, Tuple[Optional[str]]], str], new: bool = False
) -> Optional[Mapping[Union[str, Tuple[Optional[str]]], str]]:
    """Internal callback for ``nullify_links()``."""
    for key in tuple(attrs.keys()):
        if isinstance(key, tuple) and "href" in key:
            attrs[key] = "javascript:void(0)"

    return attrs


def nullify_links(text: str) -> str:
    """Given a string containing HTML, changes the ``href`` attribute of any links to "javascript:void(0)" to render
    the link useless.

    Args:
        text: The HTML string in which links should be nullified.

    Returns:
        The HTML string with all links nullified.

    """
    return bleach.linkify(text, [link_removal_callback])


def safe_fcps_emerg_html(text: str, base_url: str) -> str:
    def translate_link_attr(  # pylint: disable=unused-argument
        attrs: Mapping[Union[str, Tuple[Optional[str]]], str], new: bool = False
    ) -> Optional[Mapping[Union[str, Tuple[Optional[str]]], str]]:
        for key in tuple(attrs.keys()):
            if isinstance(key, tuple) and "href" in key:
                # Translate links that don't specify a protocol/host
                # For example, /a/b will be translated to (something like) https://fcps.edu/a/b
                attrs[key] = urllib.parse.urljoin(base_url, attrs[key])

        return attrs

    return bleach.linkify(bleach.clean(text, strip=True, tags=tags, attributes=att, css_sanitizer=css_sanitizer), [translate_link_attr])

1	import urllib.parse	1✔
2	from typing import Mapping, Optional, Tuple, Union	1✔
3
4	import bleach	1✔
5	from bleach.css_sanitizer import CSSSanitizer	1✔
6
7	tags = ["a", "abbr", "acronym", "b", "br", "blockquote", "code", "em", "hr", "i", "li", "ol", "strong", "ul", "iframe", "img", "div", "p"]	1✔
8
9	att = {	1✔
10	"acronym": ["title"],
11	"a": ["href", "title"],
12	"abbr": ["title"],
13	"iframe": ["src", "height", "width", "allowfullscreen", "frameborder"],
14	"img": ["src", "alt", "title", "style"],
15	}
16
17	css_sanitizer = CSSSanitizer(	1✔
18	allowed_css_properties=[
19	"width",
20	"height",
21	"float",
22	"margin-left",
23	"margin-right",
24	"margin-top",
25	"margin-bottom",
26	"margin",
27	"border",
28	"border-style",
29	"border-width",
30	]
31	)
32
33
34	def safe_html(txt):	1✔
35	return bleach.linkify(bleach.clean(txt, tags=tags, attributes=att, css_sanitizer=css_sanitizer))	1✔
36
37
38	def link_removal_callback( # pylint: disable=unused-argument	1✔
39	attrs: Mapping[Union[str, Tuple[Optional[str]]], str], new: bool = False
40	) -> Optional[Mapping[Union[str, Tuple[Optional[str]]], str]]:
41	"""Internal callback for ``nullify_links()``."""
42	for key in tuple(attrs.keys()):	×
43	if isinstance(key, tuple) and "href" in key:	×
44	attrs[key] = "javascript:void(0)"	×
45
46	return attrs	×
47
48
49	def nullify_links(text: str) -> str:	1✔
50	"""Given a string containing HTML, changes the ``href`` attribute of any links to "javascript:void(0)" to render
51	the link useless.
52
53	Args:
54	text: The HTML string in which links should be nullified.
55
56	Returns:
57	The HTML string with all links nullified.
58
59	"""
60	return bleach.linkify(text, [link_removal_callback])	×
61
62
63	def safe_fcps_emerg_html(text: str, base_url: str) -> str:	1✔
64	def translate_link_attr( # pylint: disable=unused-argument	×
65	attrs: Mapping[Union[str, Tuple[Optional[str]]], str], new: bool = False
66	) -> Optional[Mapping[Union[str, Tuple[Optional[str]]], str]]:
67	for key in tuple(attrs.keys()):	×
68	if isinstance(key, tuple) and "href" in key:	×
69	# Translate links that don't specify a protocol/host
70	# For example, /a/b will be translated to (something like) https://fcps.edu/a/b
71	attrs[key] = urllib.parse.urljoin(base_url, attrs[key])	×
72
73	return attrs	×
74
NEW 75	return bleach.linkify(bleach.clean(text, strip=True, tags=tags, attributes=att, css_sanitizer=css_sanitizer), [translate_link_attr])	×

tjcsl / ion / 3176824910

Source File Press 'n' to go to next uncovered line, 'b' for previous

Source File
Press 'n' to go to next uncovered line, 'b' for previous