[pbs-devel] applied: [PATCH proxmox-backup 2/2] backup_manager: use confirmation helper in wipe-disk command

[Wolfgang Bumiller]

applied, but restored the newline that was droppped with the patch

On Wed, Jun 19, 2024 at 12:15:44PM GMT, Gabriel Goller wrote:
> Use `Confirmation` helper in the wipe-disk command prompt.
> 
> Improves: 887d83cb (cli: add interactive confirmation for block device wipe, 2023-11-29)
> Cc: Markus Frank <m.frank at proxmox.com>
> Signed-off-by: Gabriel Goller <g.goller at proxmox.com>
> ---
>  src/bin/proxmox_backup_manager/disk.rs | 23 +++++++++++------------
>  1 file changed, 11 insertions(+), 12 deletions(-)
> 
> diff --git a/src/bin/proxmox_backup_manager/disk.rs b/src/bin/proxmox_backup_manager/disk.rs
> index 9c55a989..d0fdfc6f 100644
> --- a/src/bin/proxmox_backup_manager/disk.rs
> +++ b/src/bin/proxmox_backup_manager/disk.rs
> @@ -3,7 +3,7 @@ use serde_json::Value;
>  
>  use proxmox_router::{cli::*, ApiHandler, RpcEnvironment};
>  use proxmox_schema::api;
> -use std::io::{IsTerminal, Write};
> +use std::io::IsTerminal;
>  
>  use pbs_api_types::{
>      ZfsCompressionType, ZfsRaidLevel, BLOCKDEVICE_DISK_AND_PARTITION_NAME_SCHEMA,
> @@ -155,17 +155,16 @@ async fn wipe_disk(mut param: Value, rpcenv: &mut dyn RpcEnvironment) -> Result<
>  
>      // If we're on a TTY, query the user
>      if std::io::stdin().is_terminal() {
> -        println!("You are about to wipe block device {}.", param["disk"]);
> -        print!("Are you sure you want to continue? (y/N): ");
> -        let _ = std::io::stdout().flush();
> -        use std::io::{BufRead, BufReader};
> -        let mut line = String::new();
> -        match BufReader::new(std::io::stdin()).read_line(&mut line) {
> -            Ok(_) => match line.trim() {
> -                "y" | "Y" => (), // continue
> -                _ => bail!("Aborting."),
> -            },
> -            Err(err) => bail!("Failed to read line - {err}."),
> +        let confirmation = Confirmation::query_with_default(
> +            format!(
> +                "You are about to wipe block device {}. Are you sure you want to continue?",
> +                param["disk"]
> +            )
> +            .as_str(),
> +            Confirmation::No,
> +        )?;
> +        if confirmation.is_no() {
> +            bail!("Aborting.");
>          }
>      }
>  
> -- 
> 2.43.0
> 
> 
> 
> _______________________________________________
> pbs-devel mailing list
> pbs-devel at lists.proxmox.com
> https://lists.proxmox.com/cgi-bin/mailman/listinfo/pbs-devel
> 
>