From 2d307f47bd43902ee8f15751130dcbf4a4f775bf Mon Sep 17 00:00:00 2001 From: Jordan Liggitt Date: Sun, 21 Nov 2021 09:13:37 -0500 Subject: [PATCH] Return BadRequest for invalid large patch --- staging/src/k8s.io/apiserver/pkg/endpoints/handlers/patch.go | 3 +++ 1 file changed, 3 insertions(+) diff --git a/staging/src/k8s.io/apiserver/pkg/endpoints/handlers/patch.go b/staging/src/k8s.io/apiserver/pkg/endpoints/handlers/patch.go index de034aa731e..6803baaa608 100644 --- a/staging/src/k8s.io/apiserver/pkg/endpoints/handlers/patch.go +++ b/staging/src/k8s.io/apiserver/pkg/endpoints/handlers/patch.go @@ -412,6 +412,9 @@ func (p *jsonPatcher) applyJSPatch(versionedJS []byte) (patchedJS []byte, strict } patchedJS, retErr = jsonpatch.MergePatch(versionedJS, p.patchBytes) + if retErr == jsonpatch.ErrBadJSONPatch { + return nil, nil, errors.NewBadRequest(retErr.Error()) + } return patchedJS, strictErrors, retErr default: // only here as a safety net - go-restful filters content-type