Fix TikTok regex

TechAndCheck · Mar 29, 2024 · f533c1d · f533c1d
1 parent 7bf554d
commit f533c1d
Show file tree

Hide file tree

Showing 2 changed files with 3 additions and 3 deletions.
diff --git a/app/media_sources/tik_tok_media_source.rb b/app/media_sources/tik_tok_media_source.rb
@@ -42,7 +42,7 @@ def initialize(url)
   sig { params(url: String).returns(T::Boolean) }
   def self.validate_tiktok_post_url(url)
     self.valid_host_name.each do |host_name|
-      return true if /#{host_name}\/@[\w]+\/video\/[0-9]+/.match?(url)
+      return true if /https:\/\/#{host_name}\/@[\w.-]+\/video\/[0-9]+/.match?(url)
     end
 
     raise InvalidTikTokPostUrlError, "TikTok url #{url} does not have the standard url format"

diff --git a/app/models/scrape.rb b/app/models/scrape.rb
@@ -66,9 +66,9 @@ def send_notification
   def send_completion_email
     if self.user.present?
       if self.removed
-        ScrapeMailer.with(url: self.url, user: self.user).scrape_removed_email.deliver_later
+        ScrapeMailer.with(url: self.url, user: self.user, scrape_id: self.id).scrape_removed_email.deliver_later
       elsif self.error
-        ScrapeMailer.with(url: self.url, user: self.user).scrape_error_email.deliver_later
+        ScrapeMailer.with(url: self.url, user: self.user, scrape_id: self.id).scrape_error_email.deliver_later
       else
         ScrapeMailer.with(url: self.url, user: self.user).scrape_complete_email.deliver_later
       end