Skip to content
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

Delivery info have no ApproximateReceiveCount #1814

Open
F1ashhimself opened this issue Oct 24, 2023 · 1 comment
Open

Delivery info have no ApproximateReceiveCount #1814

F1ashhimself opened this issue Oct 24, 2023 · 1 comment

Comments

@F1ashhimself
Copy link

Versions:

  • Python 3.10
  • Celery 5.3.4
  • Kombu 5.3.2

After updating celery and kombu to latest versions I have stopped receiving ApproximateReceiveCount in delivery_info properties. With celery 5.3.1 and kombu 5.3.1 all works fine and properties contains ApproximateReceiveCount field.

{
    "body_encoding": "base64",
    "correlation_id": "<correlation_id_hidden>",
    "delivery_info": {
        "exchange": "",
        "routing_key": "celery",
        "sqs_message": {
            "Attributes": {
                "ApproximateReceiveCount": "1"
            },
            "Body": "<body_hidden>"
        },
        "sqs_queue": "<sqs_hidden>"
    },
    "delivery_mode": 2,
    "delivery_tag": "<delivery_tag_hidden>",
    "priority": 0,
    "reply_to": "<reply_to_hidden>"
}
@FrankK-1234
Copy link
Contributor

Hi @F1ashhimself, I'm not the expert on this issue, however it could be that this was introduced in kombu 5.3.2 due to a big change in SQS see #1726 and #1759. Although that code change still had some reference to ApproximateReceiveCount, your report show that it looks like it did not include the attribute.

That change was reverted in kombu 5.3.3, see #1799. So perhaps that version has the attribute again. I'm unable to verify that, perhaps you can do that.

We have to keep your report in mind in case another try is made #1726 as it looks like a change in behaviour.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

3 participants
@F1ashhimself @FrankK-1234 and others