harmacist/healthchecks
1
0
Fork 0
forked from GithubBackups/healthchecks
Code Issues Pull Requests Projects Releases Wiki Activity

Replace non-breaking spaces with regular spaces for cheaper SMS messages.

Browse Source
This commit is contained in:
Pēteris Caune 2017-08-10 21:50:36 +03:00
parent 31353fe24c
commit 1851cc7af3
2 changed files with 6 additions and 1 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

3
hc/api/tests/test_notify.py
View File

@ -314,6 +314,8 @@ class NotifyTestCase(BaseTestCase):
@patch("hc.api.transports.requests.request")
def test_sms(self, mock_post):
self._setup_data("sms", "+1234567890")
self.check.last_ping = now() - td(hours=2)
mock_post.return_value.status_code = 200
self.channel.notify(self.check)
@ -322,6 +324,7 @@ class NotifyTestCase(BaseTestCase):
args, kwargs = mock_post.call_args
payload = kwargs["data"]
self.assertEqual(payload["To"], "+1234567890")
self.assertFalse(u"\xa0" in payload["Body"])
# sent SMS counter should go up
self.profile.refresh_from_db()

4
hc/api/transports.py
View File

@ -11,7 +11,9 @@ from hc.lib import emails
def tmpl(template_name, **ctx):
template_path = "integrations/%s" % template_name
return render_to_string(template_path, ctx).strip()
# \xa0 is non-breaking space. It causes SMS messages to use UCS2 encoding
# and cost twice the money.
return render_to_string(template_path, ctx).strip().replace(u"\xa0", " ")
class Transport(object):