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

Thomas Lamprecht t.lamprecht at proxmox.com
Tue Feb 23 18:00:33 CET 2021


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