Expose sqlx errors directly in repo interfaces.

BoxedError conceals the exact nature of the error, which in turn prevents me from using sqlx::Error::RowNotFound to signal absences.

1 files changed, 5 insertions, 5 deletions

diff --git a/src/login/repo/tokens.rs b/src/login/repo/tokens.rs
index 2e46186..4d381ef 100644
--- a/src/login/repo/tokens.rs
+++ b/src/login/repo/tokens.rs
@@ -3,7 +3,7 @@ use sqlx::{sqlite::Sqlite, SqliteConnection, Transaction};
 use uuid::Uuid;
 
 use super::logins::{Id as LoginId, Login};
-use crate::{clock::DateTime, error::BoxedError};
+use crate::clock::DateTime;
 
 pub trait Provider {
     fn tokens(&mut self) -> Tokens;
@@ -24,7 +24,7 @@ impl<'c> Tokens<'c> {
         &mut self,
         login: &LoginId,
         issued_at: DateTime,
-    ) -> Result<String, BoxedError> {
+    ) -> Result<String, sqlx::Error> {
         let secret = Uuid::new_v4().to_string();
 
         let secret = sqlx::query_scalar!(
@@ -46,7 +46,7 @@ impl<'c> Tokens<'c> {
 
     /// Revoke a token by its secret. If there is no such token with that
     /// secret, this will succeed by doing nothing.
-    pub async fn revoke(&mut self, secret: &str) -> Result<(), BoxedError> {
+    pub async fn revoke(&mut self, secret: &str) -> Result<(), sqlx::Error> {
         sqlx::query!(
             r#"
                 delete
@@ -64,7 +64,7 @@ impl<'c> Tokens<'c> {
     /// Expire and delete all tokens that haven't been used within the expiry
     /// interval (right now, 7 days) prior to `expire_at`. Tokens that are in
     /// use within that period will be retained.
-    pub async fn expire(&mut self, expire_at: DateTime) -> Result<(), BoxedError> {
+    pub async fn expire(&mut self, expire_at: DateTime) -> Result<(), sqlx::Error> {
         // Somewhat arbitrarily, expire after 7 days.
         let expired_issue_at = expire_at - TimeDelta::days(7);
         sqlx::query!(
@@ -88,7 +88,7 @@ impl<'c> Tokens<'c> {
         &mut self,
         secret: &str,
         used_at: DateTime,
-    ) -> Result<Option<Login>, BoxedError> {
+    ) -> Result<Option<Login>, sqlx::Error> {
         // I would use `update … returning` to do this in one query, but
         // sqlite3, as of this writing, does not allow an update's `returning`
         // clause to reference columns from tables joined into the update. Two