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

[AB#43277]Transactional inbox outbox messaging for the Entitlement Service #252

Merged
merged 2 commits into from
Mar 5, 2024

Conversation

Zehelein
Copy link
Member

@Zehelein Zehelein commented Mar 4, 2024

Pull Request

Prerequisites

  • The PR is targeting the right branch (dev for features and master for releases)
  • potential release notes to the PR description added
  • potential testing notes to the PR description added
  • appropriate labels for the PR applied

Description

The Entitlement Service is now using transactional inbox message handling for reliable sending and processing of messages.

Release Notes

The Entitlement Service is now using transactional inbox message handling for reliable sending and processing of messages. Outgoing messages are stored as part of the business transaction in the outbox database table. From there they are sent via RabbitMQ. Incoming messages are received from RabbitMQ and stored in the "inbox" database table. The transactional message handlers ensure then an exactly-once message processing.

Testing Notes

Messaging should work just like before - but survive server crashes and DB outages. Please test that all the sent (synchronize claim definitions) and received messages (synchronize claim definitions finished/failed, claim set published, and subscription plan published) still work.

@Zehelein Zehelein force-pushed the feature/trx-inbox-outbox-entitlement branch from f02b73c to 9cf07e8 Compare March 4, 2024 10:54
@Zehelein Zehelein enabled auto-merge March 5, 2024 07:54
@Zehelein Zehelein merged commit 23d4c89 into dev Mar 5, 2024
6 checks passed
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