Removing constrain that makes no sense in general case (#4611)

Azure · Mar 17, 2021 · 168a79c · 168a79c
1 parent 8af0fe8
commit 168a79c
Showing 1 changed file with 6 additions and 2 deletions.
diff --git a/edgelet/iotedge/src/check/checks/hostname.rs b/edgelet/iotedge/src/check/checks/hostname.rs
@@ -86,8 +86,12 @@ impl Hostname {
         //
         // Instead, we punt on this check and assume that everything's fine if config_hostname is identical to the device hostname,
         // or starts with it.
-        if config_hostname != machine_hostname
-            && !config_hostname.starts_with(&format!("{}.", machine_hostname))
+        //
+        // Azure FQDN don't support capital letter so we lower case the hostname before doing the check.
+        if config_hostname.to_lowercase() != machine_hostname.to_lowercase()
+            && !config_hostname
+                .to_lowercase()
+                .starts_with(&format!("{}.", machine_hostname.to_lowercase()))
         {
             return Err(Context::new(format!(
             "configuration has hostname {} but device reports hostname {}.\n\