bump delta notifier for better error reporting #174
Merged
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Overview
bump delta notifier for better error reporting and (to be proven) slightly faster delta delivery.
connected issues and PRs:
None
Setup
docker-compose up -d
How to test/reproduce
Start app, do some changes check if delta's are sent. Creating a file via the file service would be a good test case
Challenges/uncertainties
It's already in use in some production apps, so I don't expect any issues. Some errors might come to light because they're now actually logged in the containers' log.
Checks PR readiness