diff options
| author | Joas Schilling <coding@schilljs.com> | 2018-06-27 13:54:06 +0300 |
|---|---|---|
| committer | Morris Jobke <hey@morrisjobke.de> | 2018-07-11 10:51:59 +0300 |
| commit | 4e99dc2318ade0ce4278c9edd28d49e4828242b5 (patch) | |
| tree | f207227a625abf9dd117a8a808fecbcd355b7e5b /core | |
| parent | 922c2c64b5d9a9f9ba8d7aa1f7c79a788875b7d6 (diff) | |
Fix "parsing" of email-addresses in comments and chat messages
\\b matches any non-word character, including \@ and \-
In order to not detect urls in the middle of email-addresses,
we need to check for white space characters and beginning of the
message instead.
Signed-off-by: Joas Schilling <coding@schilljs.com>
Diffstat (limited to 'core')
| -rw-r--r-- | core/js/public/comments.js | 4 |
1 files changed, 2 insertions, 2 deletions
diff --git a/core/js/public/comments.js b/core/js/public/comments.js index ac0bf8e0ab7..318d527b13d 100644 --- a/core/js/public/comments.js +++ b/core/js/public/comments.js @@ -21,7 +21,7 @@ * The downside: anything not ascii is excluded. Not sure how common it is in areas using different * alphabets… the upside: fake domains with similar looking characters won't be formatted as links */ - urlRegex: /(\b(https?:\/\/|([-A-Z0-9+_])*\.([-A-Z])+)[-A-Z0-9+&@#\/%?=~_|!:,.;()]*[-A-Z0-9+&@#\/%=~_|()])/ig, + urlRegex: /((\s|^)(https?:\/\/|([-A-Z0-9+_])*\.([-A-Z])+)[-A-Z0-9+&@#\/%?=~_|!:,.;()]*[-A-Z0-9+&@#\/%=~_|()])/ig, protocolRegex: /^https:\/\//, plainToRich: function(content) { @@ -39,7 +39,7 @@ return content.replace(this.urlRegex, function(url) { var hasProtocol = (url.indexOf('https://') !== -1) || (url.indexOf('http://') !== -1); if(!hasProtocol) { - url = 'https://' + url; + url = 'https://' + url.trim(); } var linkText = url.replace(self.protocolRegex, ''); |