Use same util function to get current time

tiurin · tiurin · commit 38620e3ef995 · 2025-05-28T09:59:18.000+02:00
datetime.utcnow() is deprecated and was giving a 2-hour difference result than get_current_time() for a computer running in CEST timezone.
This led to valid records being expired in bisect_events_by_record_age in stream poller.

Same function needs to be used across the code, especially when a comparison is made within same logic.
diff --git a/localstack-core/localstack/services/lambda_/event_source_mapping/pollers/dynamodb_poller.py b/localstack-core/localstack/services/lambda_/event_source_mapping/pollers/dynamodb_poller.py
@@ -7,6 +7,7 @@
 from localstack.services.lambda_.event_source_mapping.event_processor import (
     EventProcessor,
 )
+from localstack.services.lambda_.event_source_mapping.pipe_utils import get_current_time
 from localstack.services.lambda_.event_source_mapping.pollers.stream_poller import StreamPoller
 
 LOG = logging.getLogger(__name__)
@@ -109,7 +110,7 @@ def get_approximate_arrival_time(self, record: dict) -> float:
         # Optional according to AWS docs:
         # https://docs.aws.amazon.com/amazondynamodb/latest/APIReference/API_streams_StreamRecord.html
         # TODO: parse float properly if present from ApproximateCreationDateTime -> now works, compare via debug!
-        return record["dynamodb"].get("todo", datetime.utcnow().timestamp())
+        return record["dynamodb"].get("todo", get_current_time().timestamp())
 
     def format_datetime(self, time: datetime) -> str:
         return f"{time.isoformat(timespec='seconds')}Z"
