thegeneralist01/archivr
1
Fork 0
mirror of https://github.com/thegeneralist01/archivr synced 2026-05-30 08:36:47 +02:00
Code Issues Activity

Tighten social URL matching to avoid false positives

Browse source
This commit is contained in:
TheGeneralist 2026-03-31 12:10:06 +02:00
parent c988bea4aa
commit 49d6b04fac
1 changed files with 37 additions and 5 deletions
Download patch file Download diff file Expand all files Collapse all files

42
src/main.rs
View file

@ -163,23 +163,47 @@ fn determine_source(path: &str) -> Source {
return Source::X; return Source::X;
} }
if path.contains("instagram.com/") { if path.starts_with("https://instagram.com/")
|| path.starts_with("https://www.instagram.com/")
|| path.starts_with("http://instagram.com/")
|| path.starts_with("http://www.instagram.com/")
{
return Source::Instagram; return Source::Instagram;
} }
if path.contains("facebook.com/") || path.contains("fb.watch/") { if path.starts_with("https://facebook.com/")
|| path.starts_with("https://www.facebook.com/")
|| path.starts_with("http://facebook.com/")
|| path.starts_with("http://www.facebook.com/")
|| path.starts_with("https://fb.watch/")
|| path.starts_with("http://fb.watch/")
{
return Source::Facebook; return Source::Facebook;
} }
if path.contains("tiktok.com/") { if path.starts_with("https://tiktok.com/")
|| path.starts_with("https://www.tiktok.com/")
|| path.starts_with("http://tiktok.com/")
|| path.starts_with("http://www.tiktok.com/")
{
return Source::TikTok; return Source::TikTok;
} }
if path.contains("reddit.com/") || path.contains("redd.it/") { if path.starts_with("https://reddit.com/")
|| path.starts_with("https://www.reddit.com/")
|| path.starts_with("http://reddit.com/")
|| path.starts_with("http://www.reddit.com/")
|| path.starts_with("https://redd.it/")
|| path.starts_with("http://redd.it/")
{
return Source::Reddit; return Source::Reddit;
} }
if path.contains("snapchat.com/") { if path.starts_with("https://snapchat.com/")
|| path.starts_with("https://www.snapchat.com/")
|| path.starts_with("http://snapchat.com/")
|| path.starts_with("http://www.snapchat.com/")
{
return Source::Snapchat; return Source::Snapchat;
} }
} }
@ -642,6 +666,14 @@ mod tests {
url: "https://example.com/", url: "https://example.com/",
expected: Source::Other, expected: Source::Other,
}, },
TestCase {
url: "https://example.com/?redirect=instagram.com/reel/ABC123",
expected: Source::Other,
},
TestCase {
url: "https://notfacebook.com/watch?v=123456",
expected: Source::Other,
},
]; ];
for case in &other_cases { for case in &other_cases {