fix doc, line feed and ignore extra properties for the FileCredentials fileRef

Author: ThoSap

docs/cluster-connection.md

@@ -35,7 +35,7 @@ The referenced secret must be of type `kubernetes.io/basic-auth` and contain the
 
 Use this option when the credentials are mounted as a file instead of a Kubernetes Secret.
 
-### File format
+#### File format
 
 The file must contain JSON with the following fields:
 

operator/src/main/java/it/aboutbits/postgresql/core/FileRef.java

@@ -1,33 +1,33 @@
-package it.aboutbits.postgresql.core;
-
-import io.fabric8.generator.annotation.Required;
-import io.fabric8.generator.annotation.ValidationRule;
-import lombok.Getter;
-import lombok.Setter;
-import org.jspecify.annotations.NullMarked;
-
-/// A reference to a file inside the operator container.
-///
-/// This class is used wherever a CRD spec needs to point to a specific file
-/// The [#path] field identifies the file location within the container.
-///
-/// ### Example usage in a CR manifest
-///
-/// ```yaml
-///  spec:
-///     adminSecretFileRef:
-///         path: "/mnt/secrets/db-credentials.json"
-/// ```
-@Getter
-@Setter
-@NullMarked
-public class FileRef {
-    /// The path to the file.
-    /// Must not be blank.
-    @Required
-    @ValidationRule(
-            value = "self.trim().size() > 0",
-            message = "The path must not be empty."
-    )
-    private String path = "";
-}
+package it.aboutbits.postgresql.core;
+
+import io.fabric8.generator.annotation.Required;
+import io.fabric8.generator.annotation.ValidationRule;
+import lombok.Getter;
+import lombok.Setter;
+import org.jspecify.annotations.NullMarked;
+
+/// A reference to a file inside the operator container.
+///
+/// This class is used wherever a CRD spec needs to point to a specific file
+/// The [#path] field identifies the file location within the container.
+///
+/// ### Example usage in a CR manifest
+///
+/// ```yaml
+///  spec:
+///     adminSecretFileRef:
+///         path: "/mnt/secrets/db-credentials.json"
+/// ```
+@Getter
+@Setter
+@NullMarked
+public class FileRef {
+    /// The path to the file.
+    /// Must not be blank.
+    @Required
+    @ValidationRule(
+            value = "self.trim().size() > 0",
+            message = "The path must not be empty."
+    )
+    private String path = "";
+}

operator/src/main/java/it/aboutbits/postgresql/core/KubernetesService.java

@@ -1,5 +1,6 @@
 package it.aboutbits.postgresql.core;
 
+import com.fasterxml.jackson.annotation.JsonIgnoreProperties;
 import com.fasterxml.jackson.databind.ObjectMapper;
 import io.fabric8.kubernetes.client.KubernetesClient;
 import it.aboutbits.postgresql.crd.clusterconnection.ClusterConnection;
@@ -21,6 +22,9 @@
 public final class KubernetesService {
     private final ObjectMapper objectMapper;
 
+    /// The JSON file may carry more keys than we need, for example, the AWS Secrets Manager
+    /// format also has `engine`, `host`, `port` and `dbname`. Unknown keys are ignored.
+    @JsonIgnoreProperties(ignoreUnknown = true)
     private record FileCredentials(
             @Nullable String username,
             @Nullable String password