Skip to content

Commit 10289ad

Browse files
committed
Fix #reply{} return when requesting an existing manifest
The manifest machine `apply/3` function should always return a list of effects. This fixes `rabbitmq_stream_s3_log_manifest:get_manifest/1` when the manifest is already downloaded / cached in memory.
1 parent 6189a02 commit 10289ad

File tree

1 file changed

+1
-1
lines changed

1 file changed

+1
-1
lines changed

src/rabbitmq_stream_s3_log_manifest_machine.erl

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -222,7 +222,7 @@ apply(
222222
) ->
223223
case Manifests0 of
224224
#{Dir := {#manifest{} = Manifest, _UploadStatus}} ->
225-
{State0, #reply{to = Requester, response = Manifest}};
225+
{State0, [#reply{to = Requester, response = Manifest}]};
226226
#{Dir := {pending, Requesters}} ->
227227
Manifests = Manifests0#{Dir := {pending, [Requester | Requesters]}},
228228
{State0#?MODULE{manifests = Manifests}, []};

0 commit comments

Comments
 (0)