Merge branch 'main' into unused-telemetry

Author: ranweiler

src/agent/Cargo.lock

@@ -10,7 +10,7 @@ cobertura = { path = "../cobertura" }
 debuggable-module = { path = "../debuggable-module" }
 iced-x86 = "1.17"
 log = "0.4.17"
-regex = "1.0"
+regex = "1.7"
 symbolic = { version = "10.1", features = [
     "debuginfo",
     "demangle",

src/agent/coverage/Cargo.toml

@@ -12,7 +12,7 @@ goblin = "0.6.0"
 iced-x86 = "1.17"
 log = "0.4.17"
 pdb = "0.8.0"
-regex = "1.0"
+regex = "1.7"
 symbolic = { version = "10.1", features = ["debuginfo", "demangle", "symcache"] }
 thiserror = "1.0"
 

src/agent/debuggable-module/Cargo.toml

@@ -6,9 +6,9 @@ license = "MIT"
 
 [dependencies]
 anyhow = "1.0"
+clap = { version = "4.1.6", features = ["derive"] }
 lazy_static = "1.4"
-regex = "1.6"
-structopt = "0.3"
+regex = "1.7"
 thiserror = "1.0"
 
 [target.'cfg(windows)'.dependencies]

src/agent/dynamic-library/Cargo.toml

@@ -4,22 +4,22 @@
 use std::process::{Command, Stdio};
 
 use anyhow::Result;
-use structopt::StructOpt;
+use clap::Parser;
 
-#[derive(Debug, StructOpt)]
+#[derive(Parser, Debug)]
 struct Opt {
-    #[structopt(min_values = 1)]
+    #[arg(required = true, num_args = 1..)]
     argv: Vec<String>,
 
-    #[structopt(short, long)]
+    #[arg(short, long)]
     quiet: bool,
 
-    #[structopt(short, long)]
+    #[arg(short, long)]
     ld_library_path: Option<String>,
 }
 
 fn main() -> Result<()> {
-    let opt = Opt::from_args();
+    let opt = Opt::parse();
 
     let exe = &opt.argv[0];
     let mut cmd = Command::new(exe);

src/agent/dynamic-library/src/bin/dynamic-library.rs

@@ -10,5 +10,5 @@ description = "Minimal porting of features from libclusterfuzz"
 
 [dependencies]
 anyhow = "1.0"
-regex = "1.6.0"
+regex = "1.7.1"
 lazy_static = "1.4"

src/agent/libclusterfuzz/Cargo.toml

@@ -1,6 +1,5 @@
 // Rust generated by Microsoft, code originally from:
-//      https://github.com/google/clusterfuzz/blob/master/src/
-//          python/lib/clusterfuzz/stacktraces/constants.py
+//  https://github.com/google/clusterfuzz/blob/master/src/clusterfuzz/stacktraces/constants.py#:~:text=stack_frame_ignore_regexes
 //
 // Original Copyright:
 //

src/agent/libclusterfuzz/src/generated.rs

@@ -10,7 +10,7 @@ license = "MIT"
 anyhow = { version = "1.0", features = ["backtrace"] }
 async-trait = "0.1"
 downcast-rs = "1.2"
-env_logger = "0.9"
+env_logger = "0.10"
 futures = "0.3"
 log = "0.4"
 onefuzz = { path = "../onefuzz" }
@@ -25,7 +25,7 @@ storage-queue = { path = "../storage-queue" }
 tokio = { version = "1.24", features = ["full"] }
 url = { version = "2.3", features = ["serde"] }
 uuid = { version = "0.8", features = ["serde", "v4"] }
-clap = { version = "3.2.4", features = ["derive", "cargo"] }
+clap = { version = "4", features = ["derive", "cargo"] }
 reqwest-retry = { path = "../reqwest-retry" }
 onefuzz-telemetry = { path = "../onefuzz-telemetry" }
 backtrace = "0.3"

src/agent/onefuzz-agent/Cargo.toml

@@ -54,20 +54,21 @@ enum Opt {
 
 #[derive(Parser, Debug)]
 struct RunOpt {
-    #[clap(short, long = "--config", parse(from_os_str))]
+    #[arg(short, long = "config")]
     config_path: Option<PathBuf>,
+
     /// re-executes as a child process, recording stdout/stderr to files in
     /// the specified directory
-    #[clap(short, long = "--redirect-output", parse(from_os_str))]
+    #[arg(short, long = "redirect-output")]
     redirect_output: Option<PathBuf>,
 
-    #[clap(long = "--machine_id")]
+    #[arg(long = "machine_id")]
     machine_id: Option<Uuid>,
 
-    #[clap(long = "--machine_name")]
+    #[arg(long = "machine_name")]
     machine_name: Option<String>,
 
-    #[clap(long = "--reset_lock", takes_value = false, action = ArgAction::SetTrue )]
+    #[arg(long = "reset_lock", action = ArgAction::SetTrue )]
     reset_node_lock: bool,
 }
 

src/agent/onefuzz-agent/src/main.rs

@@ -15,19 +15,19 @@ arraydeque = "0.5"
 async-trait = "0.1"
 atexit = { path = "../atexit" }
 backoff = { version = "0.4", features = ["tokio"] }
-clap = "2.34"
+clap = { version = "4", features = ["cargo", "string"] }
 cobertura = { path = "../cobertura" }
 coverage = { path = "../coverage" }
-crossterm = "0.22"
-env_logger = "0.9"
+crossterm = "0.25"
+env_logger = "0.10"
 flume = "0.10"
 futures = "0.3"
 hex = "0.4"
 lazy_static = "1.4"
 log = "0.4"
 num_cpus = "1.15"
 onefuzz-file-format = { path = "../onefuzz-file-format" }
-regex = "1.6.0"
+regex = "1.7.1"
 reqwest = { version = "0.11", features = [
     "json",
     "stream",
@@ -49,7 +49,7 @@ thiserror = "1.0"
 tokio = { version = "1.24", features = ["full"] }
 tokio-util = { version = "0.7", features = ["full"] }
 tokio-stream = "0.1"
-tui = { version = "0.18", default-features = false, features = ['crossterm'] }
+tui = { version = "0.19", default-features = false, features = ['crossterm'] }
 url = { version = "2.3", features = ["serde"] }
 uuid = { version = "0.8", features = ["serde", "v4"] }
 chrono = { version = "0.4", default-features = false, features = [