[PATCH 1/2] REST: Don't error if a versioned field we would remove is absent

Konstantin Ryabitsev konstantin at linuxfoundation.org
Fri Oct 22 23:28:39 AEDT 2021


On Fri, 20 Aug 2021 at 17:59, Stephen Finucane <stephen at that.guru> wrote:
> > We remove fields that shouldn't be seen on old versions of the API.
> > This was done with `pop(field name)`, which will throw an exception
> > if the named field is absent from the data. However, sometimes if
> > a patch request is via an old API version, we hit this line without
> > ever having the field present.
> >
> > This is odd, but not harmful and we definitely shouldn't 500.
> >
> > Fixes: d944f17ec059 ("REST: Use versioning for modified responses")
> > Signed-off-by: Daniel Axtens <dja at axtens.net>
>
> Looks good to me.
>
> Reviewed-by: Stephen Finucane <stephen at that.guru>
>
> I squashed the test into this (with a tweak in the name) and applied it.

Hi, all:

Could this please be backported to stable/2.2 and maybe released as 2.2.6?

-K


More information about the Patchwork mailing list