Merge pull request #52 from theseus-rs/suppress-bytes-in-instrumentation

brianheineman · web-flow · commit 7a33c333abbe · 2024-04-05T10:09:50.000-06:00
fix: suppress bytes parameter in tracing instrumentation
diff --git a/postgresql_archive/src/archive.rs b/postgresql_archive/src/archive.rs
@@ -320,7 +320,7 @@ fn acquire_lock(out_dir: &Path) -> Result<PathBuf> {
 }
 
 /// Extracts the compressed tar [bytes](Bytes) to the [out_dir](Path).
-#[instrument]
+#[instrument(skip(bytes))]
 pub async fn extract(bytes: &Bytes, out_dir: &Path) -> Result<()> {
     let input = BufReader::new(Cursor::new(bytes));
     let decoder = GzDecoder::new(input);

Original file line number	Diff line number	Diff line change
`@@ -320,7 +320,7 @@ fn acquire_lock(out_dir: &Path) -> Result<PathBuf> {`
`320`	`320`	`}`
`321`	`321`
`322`	`322`	`/// Extracts the compressed tar [bytes](Bytes) to the [out_dir](Path).`
`323`		`-#[instrument]`
	`323`	`+#[instrument(skip(bytes))]`
`324`	`324`	`pub async fn extract(bytes: &Bytes, out_dir: &Path) -> Result<()> {`
`325`	`325`	`let input = BufReader::new(Cursor::new(bytes));`
`326`	`326`	`let decoder = GzDecoder::new(input);`