controllers/github: Reuse existing database connection in send_notification_email() call

Author: Turbo87

src/controllers/github/secret_scanning.rs

@@ -188,7 +188,7 @@ fn alert_revoke_token(
         "Active API token received and revoked (true positive)",
     );
 
-    if let Err(error) = send_notification_email(&token, alert, state) {
+    if let Err(error) = send_notification_email(&token, alert, state, &conn) {
         warn!(
             token_id = %token.id, user_id = %token.user_id, ?error,
             "Failed to send email notification",
@@ -202,9 +202,8 @@ fn send_notification_email(
     token: &ApiToken,
     alert: &GitHubSecretAlert,
     state: &AppState,
+    conn: &PgConnection,
 ) -> anyhow::Result<()> {
-    let conn = state.db_read()?;
-
     let user = User::find(&conn, token.user_id).context("Failed to find user")?;
     let Some(email) = user.email(&conn)? else {
         return Err(anyhow!("No address found"));