Skip to content
This repository was archived by the owner on Mar 11, 2025. It is now read-only.

[transfer-hook] Remove deprecated functions from cli #6525

Merged
merged 4 commits into from
Apr 3, 2024
Merged

[transfer-hook] Remove deprecated functions from cli #6525

Changes from all commits
Commits
Show all changes
4 commits
Select commit Hold shift + click to select a range
effdb23
removed allow deprecated
tonton-sol Mar 31, 2024
8aa347b
remove deprecated validators
tonton-sol Mar 31, 2024
03a97db
spelling
tonton-sol Mar 31, 2024
d03313d
added allow_all() to SignerSourceParserBuilder
tonton-sol Apr 2, 2024
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
30 changes: 13 additions & 17 deletions token/transfer-hook/cli/src/main.rs
View file
Open in desktop
Original file line number Diff line number Diff line change
@@ -1,13 +1,13 @@
#![allow(deprecated)]

pub mod meta;

use {
crate::meta::parse_transfer_hook_account_arg,
clap::{crate_description, crate_name, crate_version, Arg, Command},
solana_clap_v3_utils::{
input_parsers::{parse_url_or_moniker, pubkey_of_signer},
input_validators::{is_valid_pubkey, is_valid_signer, normalize_to_url_if_moniker},
input_parsers::{
parse_url_or_moniker, pubkey_of_signer, signer::SignerSourceParserBuilder,
},
input_validators::normalize_to_url_if_moniker,
keypair::DefaultSigner,
},
solana_client::nonblocking::rpc_client::RpcClient,
Expand All @@ -28,10 +28,6 @@ use {
std::{process::exit, rc::Rc},
};

fn clap_is_valid_pubkey(arg: &str) -> Result<(), String> {
is_valid_pubkey(arg)
}

// Helper function to calculate the required lamports for rent
async fn calculate_rent_lamports(
rpc_client: &RpcClient,
Expand Down Expand Up @@ -209,7 +205,7 @@ async fn main() -> Result<(), Box<dyn std::error::Error>> {
Arg::new("fee_payer")
.long("fee-payer")
.value_name("KEYPAIR")
.validator(|s| is_valid_signer(s))
.value_parser(SignerSourceParserBuilder::default().allow_all().build())
.takes_value(true)
.global(true)
.help("Filepath or URL to a keypair to pay transaction fee [default: client keypair]"),
Expand Down Expand Up @@ -237,7 +233,7 @@ async fn main() -> Result<(), Box<dyn std::error::Error>> {
.about("Create the extra account metas account for a transfer hook program")
.arg(
Arg::with_name("program_id")
.validator(clap_is_valid_pubkey)
.value_parser(SignerSourceParserBuilder::default().allow_pubkey().allow_file_path().build())
.value_name("TRANSFER_HOOK_PROGRAM")
.takes_value(true)
.index(1)
Expand All @@ -246,7 +242,7 @@ async fn main() -> Result<(), Box<dyn std::error::Error>> {
)
.arg(
Arg::with_name("token")
.validator(clap_is_valid_pubkey)
.value_parser(SignerSourceParserBuilder::default().allow_pubkey().allow_file_path().build())
.value_name("TOKEN_MINT_ADDRESS")
.takes_value(true)
.index(2)
Expand All @@ -265,7 +261,7 @@ async fn main() -> Result<(), Box<dyn std::error::Error>> {

Additional accounts with known fixed addresses can be passed at the command line in the format "<PUBKEY>:<ROLE>". The role must be "readonly", "writable". "readonlySigner", or "writableSigner".

Additional acounts requiring seed configurations can be defined in a configuration file using either JSON or YAML. The format is as follows:
Additional accounts requiring seed configurations can be defined in a configuration file using either JSON or YAML. The format is as follows:

```json
{
Expand Down Expand Up @@ -310,7 +306,7 @@ extraMetas:
Arg::new("mint_authority")
.long("mint-authority")
.value_name("KEYPAIR")
.validator(|s| is_valid_signer(s))
.value_parser(SignerSourceParserBuilder::default().allow_all().build())
.takes_value(true)
.global(true)
.help("Filepath or URL to mint-authority keypair [default: client keypair]"),
Expand All @@ -321,7 +317,7 @@ extraMetas:
.about("Update the extra account metas account for a transfer hook program")
.arg(
Arg::with_name("program_id")
.validator(clap_is_valid_pubkey)
.value_parser(SignerSourceParserBuilder::default().allow_pubkey().allow_file_path().build())
.value_name("TRANSFER_HOOK_PROGRAM")
.takes_value(true)
.index(1)
Expand All @@ -330,7 +326,7 @@ extraMetas:
)
.arg(
Arg::with_name("token")
.validator(clap_is_valid_pubkey)
.value_parser(SignerSourceParserBuilder::default().allow_pubkey().allow_file_path().build())
.value_name("TOKEN_MINT_ADDRESS")
.takes_value(true)
.index(2)
Expand All @@ -349,7 +345,7 @@ extraMetas:

Additional accounts with known fixed addresses can be passed at the command line in the format "<PUBKEY>:<ROLE>". The role must be "readonly", "writable". "readonlySigner", or "writableSigner".

Additional acounts requiring seed configurations can be defined in a configuration file using either JSON or YAML. The format is as follows:
Additional accounts requiring seed configurations can be defined in a configuration file using either JSON or YAML. The format is as follows:

```json
{
Expand Down Expand Up @@ -394,7 +390,7 @@ extraMetas:
Arg::new("mint_authority")
.long("mint-authority")
.value_name("KEYPAIR")
.validator(|s| is_valid_signer(s))
.value_parser(SignerSourceParserBuilder::default().allow_all().build())
.takes_value(true)
.global(true)
.help("Filepath or URL to mint-authority keypair [default: client keypair]"),
Expand Down
Loading