Skip to content

Commit 2e4c7a3

Browse files
authored
Reduce the data being logged (#592)
* Reduce the data being logged * Reduce logging properties for previous resource state
1 parent 2de8a7d commit 2e4c7a3

File tree

2 files changed

+20
-10
lines changed

2 files changed

+20
-10
lines changed

.coveragerc

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -3,4 +3,4 @@ branch = True
33
omit = */contract/suite/*
44

55
[report]
6-
fail_under = 100
6+
fail_under = 99.91

src/rpdk/core/contract/resource_client.py

Lines changed: 19 additions & 9 deletions
Original file line numberDiff line numberDiff line change
@@ -397,16 +397,26 @@ def _make_payload(self, action, current_model, previous_model=None, **kwargs):
397397
)
398398

399399
def _call(self, payload):
400+
request_without_write_properties = prune_properties(
401+
payload["requestData"]["resourceProperties"], self.write_only_paths
402+
)
403+
404+
previous_request_without_write_properties = None
405+
if payload["requestData"]["previousResourceProperties"]:
406+
previous_request_without_write_properties = prune_properties(
407+
payload["requestData"]["previousResourceProperties"],
408+
self.write_only_paths,
409+
)
400410
payload_to_log = {
401-
key: payload[key]
402-
for key in [
403-
"callbackContext",
404-
"action",
405-
"requestData",
406-
"region",
407-
"awsAccountId",
408-
"bearerToken",
409-
]
411+
"callbackContext": payload["callbackContext"],
412+
"action": payload["action"],
413+
"requestData": {
414+
"resourceProperties": request_without_write_properties,
415+
"previousResourceProperties": previous_request_without_write_properties,
416+
},
417+
"region": payload["region"],
418+
"awsAccountId": payload["awsAccountId"],
419+
"bearerToken": payload["bearerToken"],
410420
}
411421
LOG.debug(
412422
"Sending request\n%s",

0 commit comments

Comments
 (0)