[Bug] Never expiring pastas deleted immediately

[DaviDeMeo]

This code get called immediately when the page redirect to the newly created pasta
and the check at line 394 will always be true for pasta with expiration of "never"

microbin/src/main.rs

Lines 392 to 402 in 9e4940e

	pastas.retain(|p| {
	// delete the files too
	if p.expiration < timenow {
	// remove the file itself
	fs::remove_file(format!("./pasta_data/{}/{}", p.id_as_animals(), p.file));
	// and remove the containing directory
	fs::remove_dir(format!("./pasta_data/{}/", p.id_as_animals()));
	};
	p.expiration == 0 || p.expiration > timenow
	});

this should fix it and do one less comparison

    pastas.retain(|p| {
        // expiration is `never` or not reached
        if p.expiration == 0 || p.expiration > timenow {
            // keep
            true
        } else {
            // remove the file itself
            fs::remove_file(format!("./pasta_data/{}/{}", p.id_as_animals(), p.file));
            // and remove the containing directory
            fs::remove_dir(format!("./pasta_data/{}/", p.id_as_animals()));
            // remove
            false
        }
    });

[szabodanika]

It works fine for me with your change, can you give it a try? You can also deploy it on docker now B-).

Thanks for your help @DaviDeMeo, it is very appreciated!