Skip to content

Commit 0fe964d

Browse files
committed
fix(parser): set default value for etag field
1 parent 4893b2e commit 0fe964d

File tree

2 files changed

+2
-2
lines changed

2 files changed

+2
-2
lines changed

aws_lambda_powertools/utilities/parser/models/s3.py

+1-1
Original file line numberDiff line numberDiff line change
@@ -61,7 +61,7 @@ class S3Message(BaseModel):
6161
class S3EventNotificationObjectModel(BaseModel):
6262
key: str
6363
size: Optional[NonNegativeFloat] = None
64-
etag: Optional[str]
64+
etag: str = Field(default="")
6565
version_id: str = Field(None, alias="version-id")
6666
sequencer: Optional[str] = None
6767

tests/unit/parser/test_s3_notification.py

+1-1
Original file line numberDiff line numberDiff line change
@@ -53,7 +53,7 @@ def test_s3_eventbridge_notification_object_deleted_event():
5353
assert model.detail.bucket.name == raw_event["detail"]["bucket"]["name"]
5454
assert model.detail.object.key == raw_event["detail"]["object"]["key"]
5555
assert model.detail.object.size == raw_event["detail"]["object"].get("size")
56-
assert model.detail.object.etag == raw_event["detail"]["object"].get("etag")
56+
assert model.detail.object.etag == raw_event["detail"]["object"].get("etag", "")
5757
assert model.detail.object.sequencer == raw_event["detail"]["object"]["sequencer"]
5858
assert model.detail.request_id == raw_event["detail"]["request-id"]
5959
assert model.detail.requester == raw_event["detail"]["requester"]

0 commit comments

Comments
 (0)