Rename GetPublicKeys trait to GitProvider

Author: SRv6d

src/core.rs

@@ -1,4 +1,3 @@
-use async_trait::async_trait;
 use serde::{Deserialize, Serialize};
 use std::{fmt, str::FromStr};
 
@@ -22,11 +21,3 @@ impl fmt::Display for SshPublicKey {
         write!(f, "{}", self.key)
     }
 }
-
-#[async_trait]
-pub trait GetPublicKeys {
-    type Err;
-
-    /// Get the public keys of a user by their username.
-    async fn by_username(&self, username: &str) -> Result<Vec<SshPublicKey>, Self::Err>;
-}

src/github.rs

@@ -1,4 +1,4 @@
-use crate::{GetPublicKeys, SshPublicKey, USER_AGENT};
+use crate::{GitProvider, SshPublicKey, USER_AGENT};
 use async_trait::async_trait;
 use reqwest::{Client, Result, Url};
 
@@ -16,14 +16,14 @@ impl Api<'_> {
 }
 
 #[async_trait]
-impl GetPublicKeys for Api<'_> {
+impl GitProvider for Api<'_> {
     type Err = reqwest::Error;
 
     /// Get the signing keys of a user by their username.
     ///
     /// # API documentation
     /// https://docs.github.com/en/rest/users/ssh-signing-keys?apiVersion=2022-11-28#list-ssh-signing-keys-for-a-user
-    async fn by_username(&self, username: &str) -> Result<Vec<SshPublicKey>> {
+    async fn get_keys_by_username(&self, username: &str) -> Result<Vec<SshPublicKey>> {
         let url = self
             .base_url
             .join(&format!("/users/{username}/ssh_signing_keys"))
@@ -67,7 +67,7 @@ mod tests {
             base_url: server.base_url().parse().unwrap(),
             client: &client,
         };
-        let _ = api.by_username(username).await;
+        let _ = api.get_keys_by_username(username).await;
 
         mock.assert();
     }
@@ -122,7 +122,7 @@ mod tests {
             base_url: server.base_url().parse().unwrap(),
             client: &client,
         };
-        let keys = api.by_username(username).await.unwrap();
+        let keys = api.get_keys_by_username(username).await.unwrap();
 
         assert_eq!(keys, expected);
     }

src/gitlab.rs

@@ -1,4 +1,4 @@
-use crate::{GetPublicKeys, SshPublicKey, USER_AGENT};
+use crate::{GitProvider, SshPublicKey, USER_AGENT};
 use async_trait::async_trait;
 use reqwest::{Client, Result, Url};
 use serde::Deserialize;
@@ -17,14 +17,14 @@ impl Api<'_> {
 }
 
 #[async_trait]
-impl GetPublicKeys for Api<'_> {
+impl GitProvider for Api<'_> {
     type Err = reqwest::Error;
 
     /// Get the signing keys of a user by their username.
     ///
     /// # API documentation
     /// https://docs.gitlab.com/16.10/ee/api/users.html#list-ssh-keys-for-user
-    async fn by_username(&self, username: &str) -> Result<Vec<SshPublicKey>> {
+    async fn get_keys_by_username(&self, username: &str) -> Result<Vec<SshPublicKey>> {
         let url = self
             .base_url
             .join(&format!(
@@ -110,7 +110,7 @@ mod tests {
             base_url: server.base_url().parse().unwrap(),
             client: &client,
         };
-        let _ = api.by_username(username).await;
+        let _ = api.get_keys_by_username(username).await;
 
         mock.assert();
     }
@@ -170,7 +170,7 @@ mod tests {
             base_url: server.base_url().parse().unwrap(),
             client: &client,
         };
-        let keys = api.by_username(username).await.unwrap();
+        let keys = api.get_keys_by_username(username).await.unwrap();
 
         assert_eq!(keys, expected);
     }

src/lib.rs

@@ -1,10 +1,12 @@
 pub use allowed_signers::{AllowedSigner, AllowedSignersFile};
 pub use config::Config;
 pub use core::*;
+pub use provider::GitProvider;
 
 mod allowed_signers;
 pub mod cli;
 mod config;
 mod core;
 mod github;
 mod gitlab;
+mod provider;

src/provider.rs

@@ -0,0 +1,11 @@
+use crate::SshPublicKey;
+use async_trait::async_trait;
+
+/// A Git provider.
+#[async_trait]
+pub trait GitProvider {
+    type Err;
+
+    /// Get the public keys of a user by their username.
+    async fn get_keys_by_username(&self, username: &str) -> Result<Vec<SshPublicKey>, Self::Err>;
+}