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

Add handle_publish_failure callback in producer #114

Open
edoardocostantinidev opened this issue Jul 15, 2022 · 5 comments
Open

Add handle_publish_failure callback in producer #114

edoardocostantinidev opened this issue Jul 15, 2022 · 5 comments
Labels
enhancement New feature or request

Comments

@edoardocostantinidev
Copy link
Contributor

Hi,
we would like to have a bit more control over producer failures, we propose to add an optional handle_publish_failure callback in Amqpx.Gen.Producer. This callback (if implemented) would be called anytime a publish fails, arguments for it could be as follows:

If you like this proposal we, as in @primait/scrooge , can tackle this issue in our friday coding activities.

cc @michelangelomo @squaini @larrywax

@edoardocostantinidev edoardocostantinidev added the enhancement New feature or request label Jul 15, 2022
@larrywax
Copy link
Contributor

🔝

@MaeIsBad
Copy link
Member

MaeIsBad commented Aug 7, 2023

Hey 👋

I just noticed this issue. Is this still relevant?

@edoardocostantinidev
Copy link
Contributor Author

Hey, it kinda fell out of our radar since we switched initiatives, could be useful for someone but atm we don't need it :)
you can close if you want @MaeIsBad

@MaeIsBad
Copy link
Member

MaeIsBad commented Aug 8, 2023

I'll open a card in our backlog, and keep this issue open then. Feel free to ask if you need us to reprioritize this.

@MaeIsBad
Copy link
Member

MaeIsBad commented Aug 8, 2023

PLATFORM-1215 Give user more control over producer failures

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
None yet
Development

No branches or pull requests

3 participants