Skip to content
New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

#

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

Jump to bottom

fix timing errors #100

Open
wants to merge 9 commits into
base: main
Choose a base branch
from
Open

fix timing errors #100

wants to merge 9 commits into from

Conversation

tziemek
Copy link
Contributor

@tziemek tziemek commented Jul 23, 2021 •
edited
Loading

  • remove HandlerHelper.Ensure calls, so that the actual exception is caught by the messaging system to retry the message

This was referenced Jul 23, 2021
Open
Open
@tziemek tziemek marked this pull request as ready for review July 29, 2021 14:17
@tziemek
Copy link
Contributor Author

tziemek commented Jul 30, 2021

Found that one yesterday
https://docs.particular.net/nservicebus/best-practices

@tziemek
Copy link
Contributor Author

tziemek commented Aug 6, 2021

  • added MultiValidationException to ITOps
  • using https://www.nuget.org/packages/Hellang.Middleware.ProblemDetails/ for creating problem details out of statuscodes > 400 && < 600 and from MultiValidationException
  • Validation in BlackService now happening in controller instead of handler
  • Using IMessageSession directly in Controller to publish/send events/commands

# for free to join this conversation on GitHub. Already have an account? # to comment
Reviewers
No reviews
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@tziemek @tommasobertoni