[pbs-devel] applied: [PATCH proxmox-backup] api2/config/tape_backup_job: fix duplicate id parameter

Dietmar Maurer dietmar at proxmox.com
Tue Feb 23 20:27:51 CET 2021


Will fix this tomorrow - we need to set the #[fixed] attribute for id


> On 02/23/2021 6:00 PM Thomas Lamprecht <t.lamprecht at proxmox.com> wrote:
> 
>  
> On 23.02.21 17:26, Dietmar Maurer wrote:
> > applied
> 
> did you read that part:
> 
> On 23.02.21 15:54, Dominik Csapak wrote:
> > i am *really* not sure if this is the correct way @Wolfgang, is
> > there another wayt to selectively use the struct members for the
> > Updater?
> 
> 
> This makes the ID optional in the schema, which is weird for an API call
> with {id} in its url (which means that without ID this can never be reached).
> 
> So not really an ideal fix, IMO, as the API schema gets basically wrong and
> possible confusing when suggesting this non-optional parameter is optional...





More information about the pbs-devel mailing list