Skip to content

Commit b5019e6

Browse files
cquirosjChristian
authored andcommitted
Apply suggestion from @cquirosj
1 parent 6a769ac commit b5019e6

File tree

1 file changed

+2
-3
lines changed

1 file changed

+2
-3
lines changed

src/ServiceControl/MessageFailures/Api/EditFailedMessagesController.cs

Lines changed: 2 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -41,9 +41,8 @@ public async Task<ActionResult<EditRetryResponse>> Edit(string failedMessageId,
4141
if (editId != null)
4242
{
4343
logger.LogWarning("Cannot edit message {FailedMessageId} because it has already been edited", failedMessageId);
44-
// We return HTTP 200 even though the edit is being ignored. This is to keep the compatibility with older versions of ServicePulse.
45-
// Newer ServicePulse versions (starting with x.x) will handle the response's payload accordingly.
46-
return Ok(new EditRetryResponse { EditIgnored = true });
44+
// We return HTTP 200 even though the edit is being ignored. This is to keep the compatibility with older versions of ServicePulse that don't handle the payload.
45+
return Ok(new { EditIgnored = true });
4746
}
4847

4948
var failedMessage = await store.ErrorBy(failedMessageId);

0 commit comments

Comments
 (0)