Projects/mysqladm-rs
Projects
/
mysqladm-rs
12
2
Fork 0
Code Issues 36 Pull Requests Releases Activity

edit-db-perm: ensure user exists before applying changes #33

Merged
oysteikt merged 1 commits from edit-db-perm-ensure-user-exists into main 2024-08-06 23:42:19 +02:00
Conversation 0 Commits 1 Files Changed 1 +8 -1
1 changed files with 8 additions and 1 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

9
src/cli/database_command.rs
View File

@ -11,7 +11,7 @@ use crate::core::{
database_operations::{ database_operations::{
apply_permission_diffs, db_priv_field_human_readable_name, diff_permissions, yn, apply_permission_diffs, db_priv_field_human_readable_name, diff_permissions, yn,
DatabasePrivileges, DATABASE_PRIVILEGE_FIELDS, DatabasePrivileges, DATABASE_PRIVILEGE_FIELDS,
}, }, user_operations::user_exists,
}; };
// TODO: Support batch creation/dropping,showing of databases, // TODO: Support batch creation/dropping,showing of databases,
@ -519,6 +519,13 @@ pub async fn edit_permissions(
.context("Could not parse permission data from editor")? .context("Could not parse permission data from editor")?
}; };
for row in permissions_to_change.iter() {
if !user_exists(&row.user, conn).await? {
// TODO: allow user to return and correct their mistake
anyhow::bail!("User {} does not exist", row.user);
}
}
let diffs = diff_permissions(permission_data, &permissions_to_change).await; let diffs = diff_permissions(permission_data, &permissions_to_change).await;
if diffs.is_empty() { if diffs.is_empty() {