feat(sourcebundle): Add callback to handle skipped files

[szokeasaurusrex]

Add a callback to SourceBundleWriter that is called every time we skip adding a file to the bundle due to a ReadFailed error.

Also, #861 missed another spot where the ReadFailed error can cause the write function to fail; this PR fixes that.

Closes #863
Fixes #860

[github-actions]

	Messages
📖	Do not forget to update Sentry-docs with your feature once the pull request gets approved.

Generated by 🚫 dangerJS against 6fb5a49

[loewenheim]

This is an elegant way of doing it!

[szokeasaurusrex]

This is an elegant way of doing it!

Yeah, many thanks to @jan-auer for helping me to come up with this idea! We are fully backwards compatible in this way

[szokeasaurusrex]

@loewenheim I also had the idea that we could similarly add a with_filter function to add a filter callback on the SourceBundleWriter to be used with write_object, which would allow us to deprecate and eventually remove write_object_with_filter.

[loewenheim]

@loewenheim I also had the idea that we could similarly add a with_filter function to add a filter callback on the SourceBundleWriter to be used with write_object, which would allow us to deprecate and eventually remove write_object_with_filter.

It's something to think about.

[p0358]

Since #860 was closed, does that mean that sentry-cli also handles this now in some way already? What would be the behavior now when a file is to be skipped? Bail out, warning, or silent debug log? (assuming either option would now log the file name)