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

By doing the validation whether the kafka message was successfully se… #455

Merged
merged 1 commit into from Oct 16, 2023

Conversation

gklijs
Copy link
Collaborator

@gklijs gklijs commented Oct 13, 2023

…nd in the prepare commit, it happened after storing the token. Which made it very likely to miss events, this has now been fixed.
Resolves #454
The project in https://github.com/gklijs/gifcard-kafka-mongo was used to validate the solution (you do need a mvn clean install of this branch to succeed)

Copy link
Member

@smcvb smcvb left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I trust in your judgment that this change fixes the issue. Furthermore, the adjustment seems fair to me too.

…nd in the prepare commit, it happened after storing the token. Which made it very likely to miss events, this has now been fixed.
@gklijs gklijs force-pushed the make_sure_error_before_token_update branch from 8d84a03 to 79497c3 Compare October 16, 2023 12:13
@sonarcloud
Copy link

sonarcloud bot commented Oct 16, 2023

Kudos, SonarCloud Quality Gate passed!    Quality Gate passed

Bug A 0 Bugs
Vulnerability A 0 Vulnerabilities
Security Hotspot A 0 Security Hotspots
Code Smell A 0 Code Smells

85.7% 85.7% Coverage
0.0% 0.0% Duplication

@gklijs gklijs merged commit 6e8994e into master Oct 16, 2023
8 checks passed
@smcvb smcvb deleted the make_sure_error_before_token_update branch October 17, 2023 07:36
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Tracking Token Updated if Creating a Publisher Fails
2 participants