refactor(permissions): remove FromStr implementations, add ::parse methods (#25473)

The `.parse()` calls in permission code are only making it more
confusing, verbosity
is encouraged and welcome in this code even at the cost of not being
concise.

Left a couple TODOs to not use `AnyError`.

1 files changed, 1 insertions, 1 deletions

diff --git a/cli/args/flags_net.rs b/cli/args/flags_net.rs
index 57cf8d527..88ffcf0e4 100644
--- a/cli/args/flags_net.rs
+++ b/cli/args/flags_net.rs
@@ -50,7 +50,7 @@ pub fn parse(paths: Vec<String>) -> clap::error::Result<Vec<String>> {
         out.push(format!("{}:{}", host, port.0));
       }
     } else {
-      host_and_port.parse::<NetDescriptor>().map_err(|e| {
+      NetDescriptor::parse(&host_and_port).map_err(|e| {
         clap::Error::raw(clap::error::ErrorKind::InvalidValue, format!("{e:?}"))
       })?;
       out.push(host_and_port)