Skip to content

Add error callback per vhost connection#56

Closed
thisisnsh wants to merge 20 commits intobloomberg:mainfrom
thisisnsh:patch-2
Closed

Add error callback per vhost connection#56
thisisnsh wants to merge 20 commits intobloomberg:mainfrom
thisisnsh:patch-2

Conversation

@thisisnsh
Copy link
Contributor

Problem statement

  • right now error callback is for entire rabbit context, for a multi vhost setup there is no possibility to provide separate error callback

Proposed changes

  • add the possibility to add error callback per vhost when creating the connection. this error callback will take precedence over the one provided by in rabbit context. all now there is a possibility to stop getting error callback for a specific vhost by putting an empty callback during creation of connection

Remaining work

  • Unit Tests
  • Integration Tests
  • Documentation

Signed-off-by: Nishant <hadanis.singh@gmail.com>
Signed-off-by: Nishant <hadanis.singh@gmail.com>
Signed-off-by: Nishant <hadanis.singh@gmail.com>
Signed-off-by: Nishant <hadanis.singh@gmail.com>
Signed-off-by: Nishant <hadanis.singh@gmail.com>
Signed-off-by: Nishant <hadanis.singh@gmail.com>
Signed-off-by: Nishant <hadanis.singh@gmail.com>
Signed-off-by: Nishant <hadanis.singh@gmail.com>
Signed-off-by: Nishant <hadanis.singh@gmail.com>
Signed-off-by: Nishant <hadanis.singh@gmail.com>
Signed-off-by: Nishant <hadanis.singh@gmail.com>
Signed-off-by: Nishant <hadanis.singh@gmail.com>
Signed-off-by: Nishant <hadanis.singh@gmail.com>
Signed-off-by: Nishant <hadanis.singh@gmail.com>
Signed-off-by: Nishant <hadanis.singh@gmail.com>
Signed-off-by: Nishant <hadanis.singh@gmail.com>
Signed-off-by: Nishant <hadanis.singh@gmail.com>
Signed-off-by: Nishant <hadanis.singh@gmail.com>
Signed-off-by: Nishant <hadanis.singh@gmail.com>
@thisisnsh thisisnsh marked this pull request as draft January 21, 2025 04:01
@thisisnsh thisisnsh closed this Jan 4, 2026
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