From f0c214506c6dd3c837c2e93a0d60124e13e10bd0 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 Kubernetes-commit: 2d307f47bd43902ee8f15751130dcbf4a4f775bf --- pkg/endpoints/handlers/patch.go | 3 +++ 1 file changed, 3 insertions(+) diff --git a/pkg/endpoints/handlers/patch.go b/pkg/endpoints/handlers/patch.go index de034aa73..6803baaa6 100644 --- a/pkg/endpoints/handlers/patch.go +++ b/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