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

Remove temp file using timestamp path

Browse source 
Delete the temp entry at store_path/temp/<timestamp> in both
the hash-exists and success paths. Stop constructing the full filename
with extension and remove the early process::exit to de-duplicate
cleanup.
This commit is contained in:
TheGeneralist 2026-03-31 10:56:20 +02:00
parent df05687ad9
commit 7d2ab89180
Signed by: thegeneralist01
SSH key fingerprint: SHA256:pp9qddbCNmVNoSjevdvQvM5z0DHN7LTa8qBMbcMq/R4
1 changed files with 3 additions and 8 deletions
Download patch file Download diff file Expand all files Collapse all files

11
src/main.rs
View file

@ -77,7 +77,7 @@ enum Source {
// INFO: yt-dlp supports a lot of sites; so, when archiving (for example) a website, the user
// -> should be asked whether they want to archive the whole website or just the video(s) on it.
fn determine_source(path: &str) -> Source {
// INFO: Extractors' URLs can be found here:
// INFO: Extractor URLs can be found here:
// -> https://github.com/yt-dlp/yt-dlp/tree/dfc0a84c192a7357dd1768cc345d590253a14fe5/yt_dlp/extractor
// TEST: X posts can have multiple videos.
@ -272,13 +272,7 @@ fn main() -> Result<()> {
// Dir or whatever. it's midnight and my brain ain't wording/braining.
if hash_exists {
println!("File already archived.");
let _ = fs::remove_file(
store_path
.join("temp")
.join(&timestamp)
.join(format!("{timestamp}{file_extension}")),
);
process::exit(0);
let _ = fs::remove_file(store_path.join("temp").join(&timestamp));
} else {
move_temp_to_raw(
&store_path
@ -288,6 +282,7 @@ fn main() -> Result<()> {
&hash,
&store_path,
)?;
let _ = fs::remove_file(store_path.join("temp").join(&timestamp));
println!("File archived successfully.");
}