[pdm-devel] [RFC datacenter-manager 5/8] pdm-api-types: remote upid: add type field to RemoteUpid

[Thomas Lamprecht]

Am 11.11.25 um 11:50 schrieb Lukas Wagner:
> diff --git a/lib/pdm-api-types/src/remote_upid.rs b/lib/pdm-api-types/src/remote_upid.rs
> index 454c9b1f..32106e42 100644
> --- a/lib/pdm-api-types/src/remote_upid.rs
> +++ b/lib/pdm-api-types/src/remote_upid.rs
> @@ -5,19 +5,30 @@ use anyhow::{bail, Error};
>  use proxmox_schema::api_types::SAFE_ID_REGEX;
>  use proxmox_schema::{ApiType, Schema, StringSchema};
>  
> +use crate::remotes::RemoteType;
> +
>  pub const REMOTE_UPID_SCHEMA: Schema = StringSchema::new("A remote UPID")
> -    .min_length("C!UPID:N:12345678:12345678:12345678:::".len())
> +    .min_length("abc:C!UPID:N:12345678:12345678:12345678:::".len())

This shouldn't be a problem as all real UPIDs should be quite a bit longer,
but if the string would have been close to the actual realistic minimum
before, increasing the minimum length would be a breaking change, or do I
miss something?

Again, should not matter in practice.


>      .schema();