Skip to content
This repository has been archived by the owner on Jan 22, 2025. It is now read-only.

clap-utils: Forbid multiple values for --signer #34482

Merged
merged 2 commits into from
Jan 5, 2024
Merged
Show file tree
Hide file tree
Changes from all commits
Commits
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
1 change: 1 addition & 0 deletions CHANGELOG.md
Original file line number Diff line number Diff line change
Expand Up @@ -28,6 +28,7 @@ Release channels have their own copy of this changelog:
* RPC's `simulateTransaction` now returns `innerInstructions` as `json`/`jsonParsed` (#34313).
* Bigtable upload now includes entry summary data for each slot, stored in a
new `entries` table
* Forbid multiple values for the `--signer` CLI flag, forcing users to specify multiple occurrences of `--signer`, one for each signature
* Upgrade Notes
* `solana-program` and `solana-sdk` default to support for Borsh v1, with
limited backward compatibility for v0.10 and v0.9. Please upgrade to Borsh v1.
Expand Down
1 change: 1 addition & 0 deletions clap-utils/src/offline.rs
Original file line number Diff line number Diff line change
Expand Up @@ -52,6 +52,7 @@ fn signer_arg<'a, 'b>() -> Arg<'a, 'b> {
.validator(is_pubkey_sig)
.requires(BLOCKHASH_ARG.name)
.multiple(true)
.number_of_values(1)
.help(SIGNER_ARG.help)
}

Expand Down
2 changes: 1 addition & 1 deletion clap-v3-utils/src/offline.rs
Original file line number Diff line number Diff line change
Expand Up @@ -53,7 +53,7 @@ fn signer_arg<'a>() -> Arg<'a> {
.value_parser(value_parser!(PubkeySignature))
.requires(BLOCKHASH_ARG.name)
.multiple_occurrences(true)
.multiple_values(true)
.multiple_values(false)
t-nelson marked this conversation as resolved.
Show resolved Hide resolved
.help(SIGNER_ARG.help)
}

Expand Down