envoy icon indicating copy to clipboard operation
envoy copied to clipboard

deps: update com_google_cel_cpp

Open kyessenov opened this issue 2 years ago • 8 comments

Commit Message: update CEL runtime Additional Description: address fuzzing issues caused by incomplete implementation of ListKeys by returning NotImplemented since it is unlikely to be used on wrappers (only needed for equality comparison) Risk Level: low Testing: added Docs Changes: none Release Notes: none Fixes: https://github.com/envoyproxy/envoy/issues/21621

kyessenov avatar Jul 20 '22 04:07 kyessenov

CC @envoyproxy/dependency-shepherds: Your approval is needed for changes made to (bazel/.*repos.*\.bzl)|(bazel/dependency_imports\.bzl)|(api/bazel/.*\.bzl)|(.*/requirements\.txt)|(.*\.patch). envoyproxy/dependency-shepherds assignee is @phlax

:cat:

Caused by: https://github.com/envoyproxy/envoy/pull/22315 was opened by kyessenov.

see: more, trace.

Fixing upstream instead of local patch...

kyessenov avatar Jul 21 '22 00:07 kyessenov

/lgtm deps

@lizan would you mind reviewing

phlax avatar Jul 26 '22 12:07 phlax

/wait-any

phlax avatar Jul 26 '22 12:07 phlax

@lizan are you going to look at this PR? Should we re-assign?

kyessenov avatar Aug 04 '22 18:08 kyessenov

Looking for any maintainer to stamp the dep update addressing fuzzing issues here. @RyanTheOptimist

kyessenov avatar Aug 09 '22 16:08 kyessenov

/lgtm deps

RyanTheOptimist avatar Aug 09 '22 19:08 RyanTheOptimist

@lizan Can you please review this?

kyessenov avatar Aug 10 '22 19:08 kyessenov

/assign-from @envoyproxy/first-pass-reviewers

jmarantz avatar Aug 15 '22 12:08 jmarantz

@envoyproxy/first-pass-reviewers assignee is @tonya11en

:cat:

Caused by: a https://github.com/envoyproxy/envoy/pull/22315#issuecomment-1214968754 was created by @jmarantz.

see: more, trace.

@phlax can you review this?

Thanks!

jmarantz avatar Aug 15 '22 12:08 jmarantz