diff options
author | Matt Jankowski <matt@jankowski.online> | 2023-10-03 11:01:45 -0400 |
---|---|---|
committer | Claire <claire.github-309c@sitedethib.com> | 2023-10-10 13:50:10 +0200 |
commit | 7fb3ee0bc67cd7078fa7558483121dbe08202d8a (patch) | |
tree | 657b429b1b5a040179188d28e8cba30115074bf3 | |
parent | 9bd027823d91fe8cc9d3c63b19c649296e626e0d (diff) |
Dont match mention in url query string (#25656)
Co-authored-by: Claire <claire.github-309c@sitedethib.com>
-rw-r--r-- | app/models/account.rb | 6 | ||||
-rw-r--r-- | spec/models/account_spec.rb | 2 |
2 files changed, 4 insertions, 4 deletions
diff --git a/app/models/account.rb b/app/models/account.rb index 23f82a56306..3c49dd0ee7d 100644 --- a/app/models/account.rb +++ b/app/models/account.rb @@ -61,9 +61,9 @@ class Account < ApplicationRecord trust_level ) - USERNAME_RE = /[a-z0-9_]+([a-z0-9_\.-]+[a-z0-9_]+)?/i - MENTION_RE = /(?<=^|[^\/[:word:]])@((#{USERNAME_RE})(?:@[[:word:]\.\-]+[[:word:]]+)?)/i - URL_PREFIX_RE = /\Ahttp(s?):\/\/[^\/]+/ + USERNAME_RE = /[a-z0-9_]+([a-z0-9_.-]+[a-z0-9_]+)?/i + MENTION_RE = %r{(?<![=/[:word:]])@((#{USERNAME_RE})(?:@[[:word:].-]+[[:word:]]+)?)}i + URL_PREFIX_RE = %r{\Ahttp(s?)://[^/]+} include Attachmentable include AccountAssociations diff --git a/spec/models/account_spec.rb b/spec/models/account_spec.rb index dc0ca3da374..b4e8a7b3672 100644 --- a/spec/models/account_spec.rb +++ b/spec/models/account_spec.rb @@ -689,7 +689,7 @@ RSpec.describe Account, type: :model do expect(subject.match('Check this out https://medium.com/@alice/some-article#.abcdef123')).to be_nil end - xit 'does not match URL querystring' do + it 'does not match URL query string' do expect(subject.match('https://example.com/?x=@alice')).to be_nil end end |