Add is_authorized request identification into log messages (#749)

blockscout · Jan 25, 2024 · 6aa1a48 · 6aa1a48
1 parent 6031682
commit 6aa1a48
Show file tree

Hide file tree

Showing 2 changed files with 16 additions and 4 deletions.
diff --git a/eth-bytecode-db/eth-bytecode-db-server/src/services/solidity_verifier.rs b/eth-bytecode-db/eth-bytecode-db-server/src/services/solidity_verifier.rs
@@ -43,6 +43,9 @@ impl solidity_verifier_server::SolidityVerifier for SolidityVerifierService {
         let (metadata, _, request) = request.into_parts();
         super::trace_verification_request!(&request);
 
+        let is_authorized = super::is_key_authorized(&self.authorized_keys, metadata)?;
+        tracing::info!(is_authorized = is_authorized);
+
         let bytecode_type = request.bytecode_type();
         let verification_request = VerificationRequest {
             bytecode: request.bytecode,
@@ -58,7 +61,7 @@ impl solidity_verifier_server::SolidityVerifier for SolidityVerifierService {
                 .metadata
                 .map(|metadata| VerificationMetadataWrapper::from_inner(metadata).try_into())
                 .transpose()?,
-            is_authorized: super::is_key_authorized(&self.authorized_keys, metadata)?,
+            is_authorized,
         };
         let result = solidity_multi_part::verify(self.client.clone(), verification_request).await;
 
@@ -73,6 +76,9 @@ impl solidity_verifier_server::SolidityVerifier for SolidityVerifierService {
         let (metadata, _, request) = request.into_parts();
         super::trace_verification_request!(&request);
 
+        let is_authorized = super::is_key_authorized(&self.authorized_keys, metadata)?;
+        tracing::info!(is_authorized = is_authorized);
+
         let bytecode_type = request.bytecode_type();
         let verification_request = VerificationRequest {
             bytecode: request.bytecode,
@@ -85,7 +91,7 @@ impl solidity_verifier_server::SolidityVerifier for SolidityVerifierService {
                 .metadata
                 .map(|metadata| VerificationMetadataWrapper::from_inner(metadata).try_into())
                 .transpose()?,
-            is_authorized: super::is_key_authorized(&self.authorized_keys, metadata)?,
+            is_authorized,
         };
         let result =
             solidity_standard_json::verify(self.client.clone(), verification_request).await;

diff --git a/eth-bytecode-db/eth-bytecode-db-server/src/services/vyper_verifier.rs b/eth-bytecode-db/eth-bytecode-db-server/src/services/vyper_verifier.rs
@@ -43,6 +43,9 @@ impl vyper_verifier_server::VyperVerifier for VyperVerifierService {
         let (metadata, _, request) = request.into_parts();
         super::trace_verification_request!(&request);
 
+        let is_authorized = super::is_key_authorized(&self.authorized_keys, metadata)?;
+        tracing::info!(is_authorized = is_authorized);
+
         let bytecode_type = request.bytecode_type();
         let verification_request = VerificationRequest {
             bytecode: request.bytecode,
@@ -57,7 +60,7 @@ impl vyper_verifier_server::VyperVerifier for VyperVerifierService {
                 .metadata
                 .map(|metadata| VerificationMetadataWrapper::from_inner(metadata).try_into())
                 .transpose()?,
-            is_authorized: super::is_key_authorized(&self.authorized_keys, metadata)?,
+            is_authorized,
         };
         let result = vyper_multi_part::verify(self.client.clone(), verification_request).await;
 
@@ -72,6 +75,9 @@ impl vyper_verifier_server::VyperVerifier for VyperVerifierService {
         let (metadata, _, request) = request.into_parts();
         super::trace_verification_request!(&request);
 
+        let is_authorized = super::is_key_authorized(&self.authorized_keys, metadata)?;
+        tracing::info!(is_authorized = is_authorized);
+
         let bytecode_type = request.bytecode_type();
         let verification_request = VerificationRequest {
             bytecode: request.bytecode,
@@ -84,7 +90,7 @@ impl vyper_verifier_server::VyperVerifier for VyperVerifierService {
                 .metadata
                 .map(|metadata| VerificationMetadataWrapper::from_inner(metadata).try_into())
                 .transpose()?,
-            is_authorized: super::is_key_authorized(&self.authorized_keys, metadata)?,
+            is_authorized,
         };
         let result = vyper_standard_json::verify(self.client.clone(), verification_request).await;