feat: use user id instead of username

Author: vohoanglong0107

src/bors/handlers/trybuild.rs

@@ -262,7 +262,7 @@ async fn check_try_permissions<Client: RepositoryClient>(
 ) -> anyhow::Result<bool> {
     let result = if !repo
         .permissions_resolver
-        .has_permission(&author.username, PermissionType::Try)
+        .has_permission(&author.id, PermissionType::Try)
         .await
     {
         tracing::info!("Permission denied");

src/github/mod.rs

@@ -46,6 +46,7 @@ impl Display for GithubRepoName {
 
 #[derive(Debug, PartialEq)]
 pub struct GithubUser {
+    pub id: u64,
     pub username: String,
     pub html_url: Url,
 }

src/github/webhook.rs

@@ -282,6 +282,7 @@ fn parse_comment_from_pr_review(
 
 fn parse_user(user: Author) -> GithubUser {
     GithubUser {
+        id: user.id.0,
         username: user.login,
         html_url: user.html_url,
     }
@@ -381,6 +382,7 @@ mod tests {
                             name: "bors-kindergarten",
                         },
                         author: GithubUser {
+                            id: 4539057,
                             username: "Kobzol",
                             html_url: Url {
                                 scheme: "https",
@@ -424,6 +426,7 @@ mod tests {
                             name: "bors-kindergarten",
                         },
                         author: GithubUser {
+                            id: 4539057,
                             username: "Kobzol",
                             html_url: Url {
                                 scheme: "https",
@@ -467,6 +470,7 @@ mod tests {
                             name: "bors-kindergarten",
                         },
                         author: GithubUser {
+                            id: 4539057,
                             username: "Kobzol",
                             html_url: Url {
                                 scheme: "https",

src/permissions.rs

@@ -14,7 +14,7 @@ pub enum PermissionType {
 /// Decides if a GitHub user can perform various actions using the bot.
 #[async_trait]
 pub trait PermissionResolver {
-    async fn has_permission(&self, username: &str, permission: PermissionType) -> bool;
+    async fn has_permission(&self, user_id: &u64, permission: PermissionType) -> bool;
     async fn reload(&self);
 }
 
@@ -46,11 +46,11 @@ impl TeamApiPermissionResolver {
 
 #[async_trait]
 impl PermissionResolver for TeamApiPermissionResolver {
-    async fn has_permission(&self, username: &str, permission: PermissionType) -> bool {
+    async fn has_permission(&self, user_id: &u64, permission: PermissionType) -> bool {
         self.permissions
             .lock()
             .await
-            .has_permission(username, permission)
+            .has_permission(user_id, permission)
     }
 
     async fn reload(&self) {
@@ -59,15 +59,15 @@ impl PermissionResolver for TeamApiPermissionResolver {
 }
 
 pub struct UserPermissions {
-    review_users: HashSet<String>,
-    try_users: HashSet<String>,
+    review_users: HashSet<u64>,
+    try_users: HashSet<u64>,
 }
 
 impl UserPermissions {
-    fn has_permission(&self, username: &str, permission: PermissionType) -> bool {
+    fn has_permission(&self, user_id: &u64, permission: PermissionType) -> bool {
         match permission {
-            PermissionType::Review => self.review_users.contains(username),
-            PermissionType::Try => self.try_users.contains(username),
+            PermissionType::Review => self.review_users.contains(user_id),
+            PermissionType::Try => self.try_users.contains(user_id),
         }
     }
 }
@@ -90,14 +90,14 @@ async fn load_permissions(repo: &GithubRepoName) -> anyhow::Result<UserPermissio
 
 #[derive(serde::Deserialize)]
 struct UserPermissionsResponse {
-    github_users: HashSet<String>,
+    github_ids: HashSet<u64>,
 }
 
 /// Loads users that are allowed to perform try/review from the Rust Team API.
 async fn load_users_from_team_api(
     repository_name: &str,
     permission: PermissionType,
-) -> anyhow::Result<HashSet<String>> {
+) -> anyhow::Result<HashSet<u64>> {
     let permission = match permission {
         PermissionType::Review => "review",
         PermissionType::Try => "try",
@@ -112,5 +112,5 @@ async fn load_users_from_team_api(
         .json::<UserPermissionsResponse>()
         .await
         .map_err(|error| anyhow::anyhow!("Cannot deserialize users from team API: {error:?}"))?;
-    Ok(users.github_users)
+    Ok(users.github_ids)
 }

src/tests/event.rs

@@ -9,6 +9,7 @@ use crate::tests::state::{default_merge_sha, default_repo_name};
 
 fn default_user() -> GithubUser {
     GithubUser {
+        id: 1,
         username: "<user>".to_string(),
         html_url: "https://user.com".parse().unwrap(),
     }

src/tests/permissions.rs

@@ -7,7 +7,7 @@ pub struct NoPermissions;
 
 #[async_trait]
 impl PermissionResolver for NoPermissions {
-    async fn has_permission(&self, _username: &str, _permission: PermissionType) -> bool {
+    async fn has_permission(&self, _username: &u64, _permission: PermissionType) -> bool {
         false
     }
     async fn reload(&self) {}
@@ -17,7 +17,7 @@ pub struct AllPermissions;
 
 #[async_trait]
 impl PermissionResolver for AllPermissions {
-    async fn has_permission(&self, _username: &str, _permission: PermissionType) -> bool {
+    async fn has_permission(&self, _username: &u64, _permission: PermissionType) -> bool {
         true
     }
     async fn reload(&self) {}
@@ -37,7 +37,7 @@ impl Default for MockPermissions {
 
 #[async_trait]
 impl PermissionResolver for Arc<Mutex<MockPermissions>> {
-    async fn has_permission(&self, _username: &str, _permission: PermissionType) -> bool {
+    async fn has_permission(&self, _username: &u64, _permission: PermissionType) -> bool {
         false
     }
     async fn reload(&self) {

src/tests/state.rs

@@ -29,6 +29,8 @@ use crate::tests::github::{default_base_branch, PRBuilder};
 
 pub fn test_bot_user() -> GithubUser {
     GithubUser {
+        // just a random one, to reduce the chance of duplicate id
+        id: 517237103,
         username: "<test-bot>".to_string(),
         html_url: "https://test-bors.bot.com".parse().unwrap(),
     }