rgw/s3-notifications: removed logic to dump POST body to prevent printing password associated with topics

igomon-bloomberg · igomon-bloomberg · commit 334009fb7364 · 2024-05-02T09:56:55.000-04:00
Signed-off-by: Igor Gomon &lt;igomon@bloomberg.net&gt;
diff --git a/src/rgw/rgw_rest_s3.cc b/src/rgw/rgw_rest_s3.cc
@@ -5174,8 +5174,6 @@ void update_attribute_map(const std::string& input, AttributeMap& map) {
 void parse_post_action(const std::string& post_body, req_state* s)
 {
   if (post_body.size() > 0) {
-    ldpp_dout(s, 10) << "Content of POST: " << post_body << dendl;
-
     if (post_body.find("Action") != string::npos) {
       const boost::char_separator<char> sep("&");
       const boost::tokenizer<boost::char_separator<char>> tokens(post_body, sep);

Original file line number	Diff line number	Diff line change
`@@ -5174,8 +5174,6 @@ void update_attribute_map(const std::string& input, AttributeMap& map) {`
`5174`	`5174`	`void parse_post_action(const std::string& post_body, req_state* s)`
`5175`	`5175`	`{`
`5176`	`5176`	`if (post_body.size() > 0) {`
`5177`		`- ldpp_dout(s, 10) << "Content of POST: " << post_body << dendl;`
`5178`		`-`
`5179`	`5177`	`if (post_body.find("Action") != string::npos) {`
`5180`	`5178`	`const boost::char_separator<char> sep("&");`
`5181`	`5179`	`const boost::tokenizer<boost::char_separator<char>> tokens(post_body, sep);`