Re: [PATCH datacenter-manager v2 12/16] pdm-api-types: add staged_fingerprints field to NodeUrl

["Thomas Ellmenreich" <t.ellmenreich@proxmox.com>]

On Wed Aug 5, 2026 at 3:18 PM CEST, Shannon Sterz wrote:

[snip]

>  /// A node and its certificate information.
>  #[derive(Clone, Debug, Deserialize, Serialize, PartialEq)]
> +#[serde(rename_all = "kebab-case")]
>  pub struct NodeUrl {
>      /// The node address.
>      pub hostname: String,
> @@ -38,6 +46,11 @@ pub struct NodeUrl {
>      /// Certificate fingerprint.
>      #[serde(skip_serializing_if = "Option::is_none")]
>      pub fingerprint: Option<String>,
> +
> +    /// A list of staged fingerprints. If one of these is encountered while connecting to a node,
> +    /// they'll replace the main certificate fingerprint. The connection will be deemed valid.
> +    #[serde(skip_serializing_if = "Option::is_none")]
> +    pub staged_fingerprints: Option<Vec<Fingerprint>>,
>  }

I'm not sure I understand what the difference between an empty Vec and a None
value is in terms of state. If there is a difference and an empty Vec has a
special meaning, maybe this should be noted in the doccomment?

[snip