Skip to content

Commit

Permalink
firehose put-record-batch fixed to return 'RequestResponses' as list …
Browse files Browse the repository at this point in the history
…of maps with 'RecordId' as a uuid4 (#1661)
  • Loading branch information
HelmiRifai authored and whummer committed Oct 19, 2019
1 parent 33af44e commit a25cca5
Showing 1 changed file with 4 additions and 1 deletion.
5 changes: 4 additions & 1 deletion localstack/services/firehose/firehose_api.py
Original file line number Diff line number Diff line change
Expand Up @@ -247,9 +247,12 @@ def post_request():
stream_name = data['DeliveryStreamName']
records = data['Records']
put_records(stream_name, records)
request_responses = []
for i in records:
request_responses.append({'RecordId': str(uuid.uuid4())})
response = {
'FailedPutCount': 0,
'RequestResponses': []
'RequestResponses': request_responses
}
elif action == '%s.UpdateDestination' % ACTION_HEADER_PREFIX:
stream_name = data['DeliveryStreamName']
Expand Down

0 comments on commit a25cca5

Please sign in to comment.