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,
     }

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)
 }