diff options
| author | Joas Schilling <coding@schilljs.com> | 2018-06-27 12:54:06 +0200 |
|---|---|---|
| committer | Joas Schilling <coding@schilljs.com> | 2018-06-27 13:04:05 +0200 |
| commit | e1936a49e7f382d7f9ab1402a3eb48cd3cf73401 (patch) | |
| tree | 2af3b36cfa9f1038bcc03ee4e391aa570787a840 /core/js | |
| parent | 221f558f6609cdfcb9e0bf9e89e398f5940254fe (diff) | |
| download | nextcloud-server-e1936a49e7f382d7f9ab1402a3eb48cd3cf73401.tar.gz nextcloud-server-e1936a49e7f382d7f9ab1402a3eb48cd3cf73401.zip | |
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/js')
| -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, ''); |