Skip to content

Conversation

@BlaiseMuhirwa
Copy link

@BlaiseMuhirwa BlaiseMuhirwa commented Aug 22, 2025

This PR improves the SQS broker documentation to clarify a common pitfall while setting up Celery with SQS.

Problem: The current documentation shows how to URL-encode AWS credentials for the broker URL using
safequote(), but doesn't clarify that these encoded credentials should NOT be used in the predefined_queues
configuration. Doing so will throw an error that looks something like this

Unrecoverable error: ClientError("An error occurred (SignatureDoesNotMatch) when calling the GetQueueAttributes
operation: The request signature we calculated does not match the signature you provided. 
Check your AWS Secret Access Key and signing method. Consult the service documentation for details.

This docs change should make the SQS setup more reliable and intuitive for users working with predefined queues.

@BlaiseMuhirwa BlaiseMuhirwa changed the title improve docs for SQS authentication Improve Docs for SQS Authentication Aug 22, 2025
@auvipy auvipy requested a review from Copilot August 25, 2025 05:25
Copy link
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Copilot encountered an error and was unable to review this pull request. You can try again by re-requesting a review.

@Nusnus Nusnus force-pushed the bm/improve-sqs-auth-docs branch from c471e52 to 4886c54 Compare August 26, 2025 01:04
@auvipy auvipy merged commit 6506ad3 into celery:main Aug 26, 2025
6 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants