New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
delete msg - check if receipt handle is valid - fixes #324 #443
Conversation
also the correct issue ref - #324
|
||
class ReceiptHandleIsInvalid(val queueName: String, receiptHandle: String) extends ElasticMQError { | ||
val code = "ReceiptHandleIsInvalid" | ||
val message = s"""The input receipt handle "$receiptHandle" is not a valid.""" |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
val message = s"""The input receipt handle "$receiptHandle" is not a valid.""" | |
val message = s"""The receipt handle "$receiptHandle" is not valid.""" |
@@ -20,3 +20,8 @@ class MessageDoesNotExist(val queueName: String, messageId: MessageId) extends E | |||
val code = "MessageDoesNotExist" | |||
val message = s"Message does not exist: $messageId in queue: $queueName" | |||
} | |||
|
|||
class ReceiptHandleIsInvalid(val queueName: String, receiptHandle: String) extends ElasticMQError { |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
maybe simply: InvalidReceiptHandle
?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Fixed, although I left the string used in the api error response to be ReceiptHandleIsInvalid
, so it conforms to https://docs.aws.amazon.com/AWSSimpleQueueService/latest/APIReference/API_DeleteMessage.html
No description provided.