[ProtoApiScrubber] Add support for nested field paths in the field checker (#41931)

Signed-off-by: Sumit Kumar <[email protected]>

Author: sumitkmr2

source/extensions/filters/http/proto_api_scrubber/scrubbing_util_lib/field_checker.cc

@@ -4,18 +4,17 @@
 #include "source/common/protobuf/protobuf.h"
 #include "source/extensions/filters/http/proto_api_scrubber/filter_config.h"
 
+#include "absl/strings/str_join.h"
 #include "proto_processing_lib/proto_scrubber/field_checker_interface.h"
 
 namespace Envoy {
 namespace Extensions {
 namespace HttpFilters {
 namespace ProtoApiScrubber {
 
-FieldCheckResults FieldChecker::CheckField(const std::vector<std::string>&,
+FieldCheckResults FieldChecker::CheckField(const std::vector<std::string>& path,
                                            const Protobuf::Field* field) const {
-  // Currently, this method only checks the top level request/response fields and hence, the field
-  // name itself represents the field_mask.
-  const std::string field_mask = field->name();
+  const std::string field_mask = absl::StrJoin(path, ".");
 
   MatchTreeHttpMatchingDataSharedPtr match_tree;
 

source/extensions/filters/http/proto_api_scrubber/scrubbing_util_lib/field_checker.h

@@ -52,7 +52,7 @@ class FieldChecker : public FieldCheckerInterface, public Logger::Loggable<Logge
    * Currently, this method only checks the top level request/response fields. The logic for nested
    * fields will be added in the future.
    */
-  FieldCheckResults CheckField(const std::vector<std::string>&,
+  FieldCheckResults CheckField(const std::vector<std::string>& path,
                                const Protobuf::Field* field) const override;
 
   /**