fix: Adding validation for unknown field names (#223)

Author: NicoMolinaOZ

src/models/blockchain/mod.rs

@@ -11,6 +11,7 @@ pub mod stellar;
 
 /// Supported blockchain platform types
 #[derive(Debug, Clone, Serialize, Deserialize, PartialEq, Eq, Hash)]
+#[serde(deny_unknown_fields)]
 pub enum BlockChainType {
 	/// Ethereum Virtual Machine based chains
 	EVM,

src/models/config/monitor_config.rs

@@ -229,7 +229,6 @@ mod tests {
 					"contract_spec": null
 				}
 			],
-            "description": "Test monitor configuration",
             "match_conditions": {
                 "functions": [
                     {"signature": "transfer(address,uint256)"}
@@ -239,8 +238,8 @@ mod tests {
                 ],
                 "transactions": [
 					{
-						"signature": "Transfer(address,address,uint256)",
-						"status": "Success"
+						"status": "Success",
+						"expression": null
 					}
                 ]
             },
@@ -293,7 +292,6 @@ mod tests {
 					"contract_spec": null
 				}
 			],
-            "description": "Test monitor configuration",
             "match_conditions": {
                 "functions": [
                     {"signature": "transfer(address,uint256)"}
@@ -303,8 +301,8 @@ mod tests {
                 ],
                 "transactions": [
 					{
-						"signature": "Transfer(address,address,uint256)",
-						"status": "Success"
+						"status": "Success",
+						"expression": null
 					}
                 ]
             },
@@ -322,7 +320,6 @@ mod tests {
 					"contract_spec": null
 				}
 			],
-            "description": "Test monitor configuration",
             "match_conditions": {
                 "functions": [
                     {"signature": "transfer(address,uint256)"}
@@ -332,8 +329,8 @@ mod tests {
                 ],
                 "transactions": [
 					{
-						"signature": "Transfer(address,address,uint256)",
-						"status": "Success"
+						"status": "Success",
+						"expression": null
 					}
                 ]
             },

src/models/core/monitor.rs

@@ -12,6 +12,7 @@ use crate::models::blockchain::ContractSpec;
 ///   to the matched transactions before triggering the notifications
 /// - Triggers to execute when conditions are met
 #[derive(Debug, Clone, Deserialize, Serialize, PartialEq, Default)]
+#[serde(deny_unknown_fields)]
 pub struct Monitor {
 	/// Unique name identifying this monitor
 	pub name: String,
@@ -37,6 +38,7 @@ pub struct Monitor {
 
 /// Contract address with optional ABI for decoding transactions and events
 #[derive(Debug, Clone, Deserialize, Serialize, PartialEq)]
+#[serde(deny_unknown_fields)]
 pub struct AddressWithSpec {
 	/// Contract address in the network's native format
 	pub address: String,
@@ -47,6 +49,7 @@ pub struct AddressWithSpec {
 
 /// Collection of conditions that can trigger a monitor
 #[derive(Debug, Clone, Deserialize, Serialize, PartialEq, Default)]
+#[serde(deny_unknown_fields)]
 pub struct MatchConditions {
 	/// Function calls to match
 	pub functions: Vec<FunctionCondition>,
@@ -60,6 +63,7 @@ pub struct MatchConditions {
 
 /// Condition for matching contract function calls
 #[derive(Debug, Clone, Deserialize, Serialize, PartialEq)]
+#[serde(deny_unknown_fields)]
 pub struct FunctionCondition {
 	/// Function signature (e.g., "transfer(address,uint256)")
 	pub signature: String,
@@ -70,6 +74,7 @@ pub struct FunctionCondition {
 
 /// Condition for matching contract events
 #[derive(Debug, Clone, Deserialize, Serialize, PartialEq)]
+#[serde(deny_unknown_fields)]
 pub struct EventCondition {
 	/// Event signature (e.g., "Transfer(address,address,uint256)")
 	pub signature: String,
@@ -80,6 +85,7 @@ pub struct EventCondition {
 
 /// Condition for matching transaction states
 #[derive(Debug, Clone, Deserialize, Serialize, PartialEq)]
+#[serde(deny_unknown_fields)]
 pub struct TransactionCondition {
 	/// Required transaction status
 	pub status: TransactionStatus,
@@ -90,6 +96,7 @@ pub struct TransactionCondition {
 
 /// Possible transaction execution states
 #[derive(Debug, Copy, Clone, Deserialize, Serialize, PartialEq)]
+#[serde(deny_unknown_fields)]
 pub enum TransactionStatus {
 	/// Match any transaction status
 	Any,
@@ -101,6 +108,7 @@ pub enum TransactionStatus {
 
 /// Conditions that should be met prior to triggering notifications
 #[derive(Debug, Clone, Deserialize, Serialize, PartialEq)]
+#[serde(deny_unknown_fields)]
 pub struct TriggerConditions {
 	/// The path to the script
 	pub script_path: String,

src/models/core/network.rs

@@ -7,6 +7,7 @@ use crate::models::{BlockChainType, SecretValue};
 /// Defines connection details and operational parameters for a specific blockchain network,
 /// supporting both EVM and Stellar-based chains.
 #[derive(Debug, Clone, Deserialize, Serialize, PartialEq)]
+#[serde(deny_unknown_fields)]
 pub struct Network {
 	/// Type of blockchain (EVM, Stellar, etc)
 	pub network_type: BlockChainType,
@@ -44,6 +45,7 @@ pub struct Network {
 
 /// RPC endpoint configuration with load balancing weight
 #[derive(Debug, Clone, Deserialize, Serialize, PartialEq)]
+#[serde(deny_unknown_fields)]
 pub struct RpcUrl {
 	/// Type of RPC endpoint (e.g. "rpc")
 	pub type_: String,

src/models/core/trigger.rs

@@ -4,6 +4,7 @@ use serde::{Deserialize, Serialize};
 
 /// Configuration for actions to take when monitored conditions are met.
 #[derive(Debug, Clone, Deserialize, Serialize, PartialEq)]
+#[serde(deny_unknown_fields)]
 pub struct Trigger {
 	/// Unique name identifying this trigger
 	pub name: String,
@@ -18,6 +19,7 @@ pub struct Trigger {
 /// Supported trigger action types
 #[derive(Debug, Clone, Deserialize, Serialize, PartialEq)]
 #[serde(rename_all = "lowercase")]
+#[serde(deny_unknown_fields)]
 pub enum TriggerType {
 	/// Send notification to Slack
 	Slack,
@@ -35,6 +37,7 @@ pub enum TriggerType {
 
 /// Notification message fields
 #[derive(Debug, Clone, Deserialize, Serialize, PartialEq)]
+#[serde(deny_unknown_fields)]
 pub struct NotificationMessage {
 	/// Notification title or subject
 	pub title: String,
@@ -44,6 +47,7 @@ pub struct NotificationMessage {
 
 /// Type-specific configuration for triggers
 #[derive(Debug, Clone, Deserialize, Serialize, PartialEq)]
+#[serde(deny_unknown_fields)]
 #[serde(untagged)]
 pub enum TriggerTypeConfig {
 	/// Slack notification configuration

src/models/security/secret.rs

@@ -117,6 +117,7 @@ pub async fn get_vault_client() -> SecurityResult<&'static VaultType> {
 /// All variants implement `ZeroizeOnDrop` to ensure secure memory cleanup.
 #[derive(Debug, Clone, Serialize, ZeroizeOnDrop)]
 #[serde(tag = "type", content = "value")]
+#[serde(deny_unknown_fields)]
 pub enum SecretValue {
 	/// A plain text secret value
 	Plain(SecretString),