feat: Enhancing debug information for HttpDataSource transfers

[zub4t]

What this PR changes/adds

Appends the reason for the error to the message that will be sent to the user when the transfer fails

Why it does that

To improve the user experience by returning some feedback when the StreamResult has no message associated with it

Linked Issue(s)

Closes #4280

[codecov-commenter]

⚠️ Please install the to ensure uploads and comments are reliably processed by Codecov.

Codecov Report

Attention: Patch coverage is 50.00000% with 1 line in your changes missing coverage. Please review.

Project coverage is 75.20%. Comparing base (7f20ba5) to head (f23e621).
Report is 334 commits behind head on main.

Files	Patch %	Lines
...onnector/dataplane/spi/pipeline/StreamFailure.java	50.00%	0 Missing and 1 partial ⚠️

❗ Your organization needs to install the Codecov GitHub app to enable full functionality.

Additional details and impacted files

@@            Coverage Diff             @@
##             main    #4303      +/-   ##
==========================================
+ Coverage   71.74%   75.20%   +3.45%     
==========================================
  Files         919     1052     +133     
  Lines       18457    21116    +2659     
  Branches     1037     1181     +144     
==========================================
+ Hits        13242    15880    +2638     
+ Misses       4756     4720      -36     
- Partials      459      516      +57     

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

[ndr-brt]

unit tests should stay as close as possible to the code, I wouldn't add them on PipelineServiceImpl nor on ParallelSink but on StreamFailure directly.

[zub4t]

unit tests should stay as close as possible to the code, I wouldn't add them on PipelineServiceImpl nor on ParallelSink but on StreamFailure directly.

I didn't do this because there is no test class dedicated to the StreamFailure class, even for most of the classes within the :spi:data-plane:data-plane-spi module there are no specific test classes. So should I create a StreamFailureTest class in this module?

[ndr-brt]

unit tests should stay as close as possible to the code, I wouldn't add them on PipelineServiceImpl nor on ParallelSink but on StreamFailure directly.

I didn't do this because there is no test class dedicated to the StreamFailure class, even for most of the classes within the :spi:data-plane:data-plane-spi module there are no specific test classes. So should I create a StreamFailureTest class in this module?

Yes, there wasn't because it was not necessary, but now it will become

[ndr-brt]

please note that other tests around the codebase are broken

[jimmarino]

@zub4t Tests are failing, please fix them.

[ndr-brt]

LGTM