Skip to content

Change createJsonFormatter factory return type to StatusOr#45831

Open
yanavlasov wants to merge 2 commits into
envoyproxy:mainfrom
yanavlasov:remove-exceptions-6
Open

Change createJsonFormatter factory return type to StatusOr#45831
yanavlasov wants to merge 2 commits into
envoyproxy:mainfrom
yanavlasov:remove-exceptions-6

fix build

217c874
Select commit
Loading
Failed to load commit list.
CI (Envoy) / Envoy/Checks succeeded Jun 25, 2026 in 47m 17s

Envoy/Checks (success)

Check has finished

Details

Check run finished (success ✔️)

The check run can be viewed here:

Envoy/Checks (pr/45831/main@217c874)

Check started by

Request (pr/45831/main@217c874)

yanavlasov @yanavlasov 217c874 #45831 merge main@df2c77d

Change createJsonFormatter factory return type to StatusOr

Prep for removing C++ exceptions from that code path. Noop at this point.

Risk Level: low, noop
Testing: unit tests
Docs Changes: no
Release Notes: no
Platform Specific Features: no

Environment

Request variables

Key Value
ref f716d79
sha 217c874
pr 45831
base-sha df2c77d
actor yanavlasov @yanavlasov
message Change createJsonFormatter factory return type to StatusOr...
started 1782346514.876376
target-branch main
trusted false
Build image

Container image/s (as used in this CI run)

Key Value
default docker.io/envoyproxy/envoy-build:v0.1.6
mobile docker.io/envoyproxy/envoy-build:mobile-v0.1.6
Version

Envoy version (as used in this CI run)

Key Value
major 1
minor 39
patch 0
dev true