diff options
| -rw-r--r-- | _pytest/test_linkifytext.py | 9 | ||||
| -rw-r--r-- | wee_slack.py | 8 |
2 files changed, 13 insertions, 4 deletions
diff --git a/_pytest/test_linkifytext.py b/_pytest/test_linkifytext.py index f9da3f9..010a48b 100644 --- a/_pytest/test_linkifytext.py +++ b/_pytest/test_linkifytext.py @@ -4,3 +4,12 @@ from wee_slack import linkify_text # linkify_text('@ryan') # assert False + + +def test_linkifytext_does_partial_html_entity_encoding(mock_weechat, realish_eventrouter): + team = realish_eventrouter.teams.values()[0] + channel = team.channels.values()[0] + + text = linkify_text('& < > \' "', team, channel) + + assert text == '& < > \' "' diff --git a/wee_slack.py b/wee_slack.py index 680ff6b..019ba5f 100644 --- a/wee_slack.py +++ b/wee_slack.py @@ -2541,11 +2541,11 @@ def linkify_text(message, team, channel): .replace('\x1D', '_') .replace('\x1F', config.map_underline_to) # Escape chars that have special meaning to Slack. Note that we do not - # (and should not) perform a full URL escaping here. + # (and should not) perform full HTML entity-encoding here. # See https://api.slack.com/docs/message-formatting for details. - .replace('<', '<') - .replace('>', '>') - .replace('&', '&') + .replace('&', '&') + .replace('<', '<') + .replace('>', '>') .split(' ')) for item in enumerate(message): targets = re.match('^\s*([@#])([\w.-]+[\w. -])(\W*)', item[1]) |