Make pedantic lints default and fix them

... except for the const unwraps.

.github/workflows/rust.yml

@@ -18,7 +18,7 @@ jobs:
         components: clippy
     - name: Run cargo-deny
       uses: EmbarkStudios/cargo-deny-action@v2
-    - run: cargo clippy -- -W clippy::pedantic
+    - run: cargo clippy
     - run: cargo fmt --all --check
     - run: cargo test --all-targets
     - run: WASTEBIN_BASE_URL="http://127.0.0.1:8080/wastebin" cargo test  # port is not relevant

Cargo.toml

@@ -52,6 +52,7 @@ if_not_else = "deny"
 items_after_statements = "deny"
 mut_mut = "deny"
 panic = "deny"
+pedantic = "deny"
 print_stdout = "deny"
 similar_names = "deny"
 unicode_not_nfc = "deny"

src/pages.rs

@@ -69,6 +69,7 @@ impl std::fmt::Display for Expiration {
     }
 }
 
+#[allow(clippy::unwrap_used)]
 const EXPIRATION_OPTIONS: [(&str, Expiration); 8] = [
     ("never", Expiration::None),
     ("10 minutes", Expiration::Time(NonZero::new(600).unwrap())),

src/routes/paste.rs

@@ -126,7 +126,7 @@ async fn get_html(
         .transpose()
         .map_err(|err| Error::CookieParsing(err.to_string()))?
         .zip(entry.uid)
-        .map_or(false, |(user_uid, owner_uid)| user_uid == owner_uid);
+        .is_some_and(|(user_uid, owner_uid)| user_uid == owner_uid);
 
     if let Some(html) = state.cache.get(&key) {
         tracing::trace!(?key, "found cached item");
@@ -263,7 +263,7 @@ pub async fn delete(
         .transpose()
         .map_err(|err| Error::CookieParsing(err.to_string()))?
         .zip(uid)
-        .map_or(false, |(user_uid, db_uid)| user_uid == db_uid);
+        .is_some_and(|(user_uid, db_uid)| user_uid == db_uid);
 
     if !can_delete {
         Err(Error::Delete)?;