Apply suggestions from code review

Co-authored-by: Techassi <[email protected]>

Author: sbernauer

crates/stackable-operator/src/commons/authentication/ldap.rs

@@ -35,7 +35,7 @@ pub enum Error {
 )]
 #[serde(rename_all = "camelCase")]
 pub struct AuthenticationProvider {
-    /// Host of the LDAP server, for example: `my.ldap.server`.
+    /// Host of the LDAP server, for example: `my.ldap.server` or `127.0.0.1`.
     pub hostname: Host,
 
     /// Port of the LDAP server. If TLS is used defaults to 636 otherwise to 389.

crates/stackable-operator/src/commons/authentication/oidc.rs

@@ -42,7 +42,7 @@ pub enum Error {
 )]
 #[serde(rename_all = "camelCase")]
 pub struct AuthenticationProvider {
-    /// Host of the identity provider, e.g. `my.keycloak.corp`.
+    /// Host of the identity provider, e.g. `my.keycloak.corp` or `127.0.0.1`.
     hostname: Host,
 
     /// Port of the identity provider. If TLS is used defaults to 443,

crates/stackable-operator/src/commons/networking.rs

@@ -93,7 +93,7 @@ impl TryFrom<String> for Host {
 
 impl From<Host> for String {
     fn from(value: Host) -> Self {
-        format!("{value}")
+        value.to_string()
     }
 }