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

Current implementation does not support SQS configuration broker transport options #98

Open
JurgenFiGO opened this issue Dec 2, 2021 · 0 comments

Comments

@JurgenFiGO
Copy link

Currently, configuration for broker_transport_options are being limited by the following map:

BROKER_TRANSPORT_OPTIONS_MAP = {
    'visibility_timeout': int,
    'max_retries': int,
}

However, for SQS, there are other options required such as region. Because these are not mapped, they are ignored and we get an error further down the line as this information is missing.

It would not be very sustainable to support every option required by every (supported) provider, but what is the potential risk of removing this mapping in its entirety? Or is there another way to configure this that I missed?

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

No branches or pull requests

1 participant