fix(sqs): don't crash on multiple predefined queues with aws sts session

[iBluemind]

Hi,

I discovered a crash that occurs when using predefined queues with AWS SQS by AWS STS authentication, as described in this documentation.

Even though multiple predefined queues are configured like the example below, only one queue is currently usable.
(If you use queue-2 first, queue-1 becomes unavailable afterward.)

transport_options = {
    'predefined_queues': {
        'queue-1': {
            'url': 'https://sqs.ap-northeast-2.amazonaws.com/xxx/aaa'
        },
        'queue-2': {
            'url': 'https://sqs.ap-northeast-2.amazonaws.com/xxx/bbb'
        }
    },
    'sts_role_arn': 'arn:aws:iam::<xxx>:role/STSTest'
}

The issue arises because the Channel._predefined_queue_clients variable is not updated, which prevents switching to another queue once one has been used.

Thanks

[jinsung-kang]

LGTM

[devEunjaeYong]

LGTM

[codecov]

Codecov Report

Attention: Patch coverage is 90.00000% with 1 line in your changes missing coverage. Please review.

Project coverage is 81.50%. Comparing base (c5e25d2) to head (9d90b5c).
Report is 1 commits behind head on main.

Files with missing lines	Patch %	Lines
kombu/transport/SQS.py	90.00%	0 Missing and 1 partial ⚠️

Additional details and impacted files

@@           Coverage Diff           @@
##             main    #2224   +/-   ##
=======================================
  Coverage   81.50%   81.50%           
=======================================
  Files          77       77           
  Lines        9522     9524    +2     
  Branches     1151     1152    +1     
=======================================
+ Hits         7761     7763    +2     
  Misses       1569     1569           
  Partials      192      192           

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.