Merge #1442: Do not reveal the API token in logs

josecelano · josecelano · commit 8a36e452caee · 2025-04-11T16:58:45.000+01:00
4205c71 fix: [#1441] do not reveal API token in logs (Jose Celano) Pull request description: Do not reveal the API token in logs. ACKs for top commit: josecelano: ACK 4205c71 Tree-SHA512: 0636fb5466bca560216410e7b386db50c254a6bc99d8e9af950c483b1275b5f845fc1dc3718c38378e1f04b9a5b666dbdd95224895fe34c3d1041f8f964904ab
diff --git a/src/container.rs b/src/container.rs
@@ -38,7 +38,7 @@ pub struct AppContainer {
 }
 
 impl AppContainer {
-    #[instrument(skip())]
+    #[instrument(skip(configuration))]
     pub fn initialize(configuration: &Configuration) -> AppContainer {
         // Configuration
 

Original file line number	Diff line number	Diff line change
`@@ -38,7 +38,7 @@ pub struct AppContainer {`
`38`	`38`	`}`
`39`	`39`
`40`	`40`	`impl AppContainer {`
`41`		`- #[instrument(skip())]`
	`41`	`+ #[instrument(skip(configuration))]`
`42`	`42`	`pub fn initialize(configuration: &Configuration) -> AppContainer {`
`43`	`43`	`// Configuration`
`44`	`44`