cleanup username and apikey structs

vtse-common/src/user.rs

@@ -1,14 +1,24 @@
 use serde::Deserialize;
 use uuid::Uuid;
 
-#[derive(Clone, Eq, PartialEq, Ord, PartialOrd, Hash, Debug, Default, Deserialize, sqlx::Type)]
+#[derive(Clone, Eq, PartialEq, Ord, PartialOrd, Hash, Debug, Default, Deserialize)]
-#[sqlx(transparent)]
+pub struct ApiKey(Uuid);
-pub struct ApiKey(pub Uuid);
 
-#[derive(Clone, Eq, PartialEq, Ord, PartialOrd, Hash, Debug, Default, Deserialize, sqlx::Type)]
+impl ApiKey {
-#[sqlx(transparent)]
+    pub fn inner(&self) -> &Uuid {
+        &self.0
+    }
+}
+
+#[derive(Clone, Eq, PartialEq, Ord, PartialOrd, Hash, Debug, Default, Deserialize)]
 pub struct Username(String);
 
+impl Username {
+    pub fn inner(&self) -> &str {
+        &self.0
+    }
+}
+
 #[derive(Clone, Eq, PartialEq, Ord, PartialOrd, Hash, Debug, Default, Deserialize)]
 pub struct Password(String);
 

vtse-server/src/operations.rs

@@ -1,6 +1,5 @@
 use serde::Deserialize;
 use vtse_common::stock::StockName;
-
 use vtse_common::user::{ApiKey, Password, Username};
 
 #[derive(Deserialize, Debug, PartialEq)]

vtse-server/src/state.rs

@@ -72,7 +72,7 @@ impl AppState {
                 "SELECT (user_id) FROM user_api_keys
                 JOIN api_keys ON user_api_keys.key_id = api_keys.key_id
                 WHERE key = $1",
-                api_key.0
+                api_key.inner()
             )
             .fetch_optional(pool)
             .await?;
@@ -98,7 +98,7 @@ impl AppState {
             "INSERT INTO users
             (username, pwhash_data)
             VALUES ($1, $2)",
-            username as Username,
+            username.inner(),
             salted_password.as_ref(),
         )
         .execute(pool)
@@ -156,7 +156,7 @@ impl AppState {
         let result = query!(
             "SELECT user_id, pwhash_data FROM users
             WHERE username = $1",
-            username as &Username
+            username.inner()
         )
         .fetch_one(pool)
         .await?;

vtse-server/src/user.rs

@@ -5,19 +5,18 @@ use sodiumoxide::crypto::pwhash::{
 use std::convert::TryFrom;
 use vtse_common::user::Password;
 
-#[derive(sqlx::Type)]
-#[sqlx(transparent)]
 pub(crate) struct SaltedPassword(HashedPassword);
 
 impl TryFrom<Password> for SaltedPassword {
     type Error = ();
 
     fn try_from(password: Password) -> Result<Self, Self::Error> {
-        Ok(Self(pwhash(
+        pwhash(
             password.as_bytes(),
             OPSLIMIT_INTERACTIVE,
             MEMLIMIT_INTERACTIVE,
-        )?))
+        )
+        .map(Self)
     }
 }