raol / amazon-sqs-net-extended-client-lib

Extension to Amazon SQS that adds support for sending and receiving messages greater than 256K
Apache License 2.0
39 stars 33 forks source link

ChangeMessageVisibilityAsync() Fails for messages from S3 #15

Closed nrcmkoh closed 2 years ago

nrcmkoh commented 5 years ago

Hi there, I am getting an exception when I execute this method for messages from S3. It works fine for messages that aren't serialized to S3.

My code looks like this:

            try {
                var request = new ChangeMessageVisibilityRequest(queue, receiptHandle, 5);
                var result = sqsClient.ChangeMessageVisibilityAsync(request).Result;
            } catch (MessageNotInflightException ex) {
                _logger.LogError(ex, ex.Message);
                throw;
            } catch (ReceiptHandleIsInvalidException ex) {
                _logger.LogError(ex, ex.Message);
                throw;
            } catch (Exception ex) {
                // Error is getting caught here
                throw;
            }

I suspect that the receipt handle is unrecognizable to this method when the message is in S3.

raol commented 5 years ago

Thanks. I'll try to look into this in the near future.

csheets-omnitracs commented 3 years ago

just wondering if there is an update on this issue? -- I am evaluating the extended client for large SQSS messages and we use ChangeMessageVisibilityAsync method often and so have run into this