From 371ef3ce95549856a1db7cc72603d7b5ed8bbbf8 Mon Sep 17 00:00:00 2001 From: Moxie Marlinspike Date: Wed, 9 Dec 2015 20:07:14 -0800 Subject: [PATCH] Strip whitespace in addition to parentheses for outgoing SMS Fixes #4833 // FREEBIE --- src/org/thoughtcrime/securesms/jobs/SmsSendJob.java | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/src/org/thoughtcrime/securesms/jobs/SmsSendJob.java b/src/org/thoughtcrime/securesms/jobs/SmsSendJob.java index ff3be66baf..0e8f3a7b19 100644 --- a/src/org/thoughtcrime/securesms/jobs/SmsSendJob.java +++ b/src/org/thoughtcrime/securesms/jobs/SmsSendJob.java @@ -95,7 +95,7 @@ public class SmsSendJob extends SendJob { // stripSeparator() to rigid? what will people try to send here? - see discussion on #3099 //recipient = PhoneNumberUtils.stripSeparators(recipient); // minimum required to fix the issue - brackets are no dialable chars... - recipient = recipient.replaceAll("[\\(\\)]", ""); + recipient = recipient.replaceAll("[\\(\\)\\s]", ""); // NOTE 11/04/14 -- There's apparently a bug where for some unknown recipients // and messages, this will throw an NPE. We have no idea why, so we're just