correct onvif filters

agent/Cargo.toml

@@ -1,6 +1,6 @@
 [package]
 name = "agent"
-version = "0.10.12"
+version = "0.10.13"
 authors = ["Kate Goldenring <[email protected]>", "<[email protected]>"]
 edition = "2018"
 rust-version = "1.68.1"

controller/Cargo.toml

@@ -1,6 +1,6 @@
 [package]
 name = "controller"
-version = "0.10.12"
+version = "0.10.13"
 authors = ["<[email protected]>", "<[email protected]>"]
 edition = "2018"
 rust-version = "1.68.1"

deployment/helm/Chart.yaml

@@ -16,9 +16,9 @@ type: application
 # This is the chart version. This version number should be incremented each time you make changes
 # to the chart and its templates, including the app version.
 # Versions are expected to follow Semantic Versioning (https://semver.org/)
-version: 0.10.12
+version: 0.10.13
 
 # This is the version number of the application being deployed. This version number should be
 # incremented each time you make changes to the application. Versions are not expected to
 # follow Semantic Versioning. They should reflect the version the application is using.
-appVersion: 0.10.12
+appVersion: 0.10.13

discovery-handler-modules/debug-echo-discovery-handler/Cargo.toml

@@ -1,6 +1,6 @@
 [package]
 name = "debug-echo-discovery-handler"
-version = "0.10.12"
+version = "0.10.13"
 authors = ["Kate Goldenring <[email protected]>"]
 edition = "2018"
 rust-version = "1.68.1"

discovery-handler-modules/onvif-discovery-handler/Cargo.toml

@@ -1,6 +1,6 @@
 [package]
 name = "onvif-discovery-handler"
-version = "0.10.12"
+version = "0.10.13"
 authors = ["Kate Goldenring <[email protected]>"]
 edition = "2018"
 rust-version = "1.68.1"

discovery-handler-modules/opcua-discovery-handler/Cargo.toml

@@ -1,6 +1,6 @@
 [package]
 name = "opcua-discovery-handler"
-version = "0.10.12"
+version = "0.10.13"
 authors = ["Kate Goldenring <[email protected]>"]
 edition = "2018"
 rust-version = "1.68.1"

discovery-handler-modules/udev-discovery-handler/Cargo.toml

@@ -1,6 +1,6 @@
 [package]
 name = "udev-discovery-handler"
-version = "0.10.12"
+version = "0.10.13"
 authors = ["Kate Goldenring <[email protected]>"]
 edition = "2018"
 rust-version = "1.68.1"

discovery-handlers/debug-echo/Cargo.toml

@@ -1,6 +1,6 @@
 [package]
 name = "akri-debug-echo"
-version = "0.10.12"
+version = "0.10.13"
 authors = ["Kate Goldenring <[email protected]>"]
 edition = "2018"
 rust-version = "1.68.1"

discovery-handlers/onvif/Cargo.toml

@@ -1,6 +1,6 @@
 [package]
 name = "akri-onvif"
-version = "0.10.12"
+version = "0.10.13"
 authors = ["Kate Goldenring <[email protected]>"]
 edition = "2018"
 rust-version = "1.68.1"

discovery-handlers/onvif/src/discovery_handler.rs

@@ -168,6 +168,7 @@ async fn apply_filters(
     if util::execute_filter(
         discovery_handler_config.ip_addresses.as_ref(),
         &ip_address_as_vec,
+        |scope, pattern| scope == pattern,
     ) {
         return None;
     }
@@ -177,6 +178,7 @@ async fn apply_filters(
     if util::execute_filter(
         discovery_handler_config.mac_addresses.as_ref(),
         &mac_address_as_vec,
+        |scope, pattern| scope.to_lowercase() == pattern.to_lowercase(),
     ) {
         return None;
     }
@@ -583,4 +585,64 @@ mod tests {
             .await
             .is_none());
     }
+
+    #[tokio::test]
+    async fn test_apply_filters_include_mac_exist_different_cases() {
+        let mock_uri = "device_uri";
+        let mock_ip = "mock.ip";
+        let mock_mac = "MocK:Mac";
+
+        let mut mock = MockOnvifQuery::new();
+        configure_scenario(
+            &mut mock,
+            Some(IpAndMac {
+                mock_uri,
+                mock_ip,
+                mock_mac,
+            }),
+        );
+
+        let onvif_config = OnvifDiscoveryDetails {
+            ip_addresses: None,
+            mac_addresses: Some(FilterList {
+                action: FilterType::Include,
+                items: vec![mock_mac.to_uppercase()],
+            }),
+            scopes: None,
+            discovery_timeout_seconds: 1,
+        };
+        let instance = apply_filters(&onvif_config, mock_uri, &mock).await.unwrap();
+
+        assert_eq!(expected_device(mock_uri, mock_ip, mock_mac), instance);
+    }
+
+    #[tokio::test]
+    async fn test_apply_filters_exclude_mac_exist_different_cases() {
+        let mock_uri = "device_uri";
+        let mock_ip = "mock.ip";
+        let mock_mac = "MocK:Mac";
+
+        let mut mock = MockOnvifQuery::new();
+        configure_scenario(
+            &mut mock,
+            Some(IpAndMac {
+                mock_uri,
+                mock_ip,
+                mock_mac,
+            }),
+        );
+
+        let onvif_config = OnvifDiscoveryDetails {
+            ip_addresses: None,
+            mac_addresses: Some(FilterList {
+                action: FilterType::Exclude,
+                items: vec![mock_mac.to_uppercase()],
+            }),
+            scopes: None,
+            discovery_timeout_seconds: 1,
+        };
+        assert!(apply_filters(&onvif_config, mock_uri, &mock)
+            .await
+            .is_none());
+    }
 }