envoyproxy / envoyproxy/envoy

Remove unused methods and exceptions from JsonObject

Open
#14,893 0 comments 0 reactions 0 assignees View on GitHub
help wanted tech debt
Dominant language
C++
Stars
28.9k
Forks
5.6k
Avg merge
1d 22h
Merged PRs (30d)
430

Description

1. Remove instance of `loadFromString` in header formatter and replace with protobuf . https://github.com/envoyproxy/envoy/blob/e073b325890542a3ce3bbc4eefb89a09f390f27e/source/common/router/header_formatter.cc#L61

2. Remove unused methods and the unsafe code that can throw exceptions such as the get[Type] methods that don't have defaults. https://github.com/envoyproxy/envoy/blob/0d5b470097d5e3645759e43414db8a7104aafb2e/include/envoy/json/json_object.h#L114

Follow-up to https://github.com/envoyproxy/envoy/pull/14467

Contributor guide

Open the contributing guide

Assessment

This issue has not been assessed yet.

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.