Skip to content

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
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.

1 participant