Revert "Add "unsafe" AsyncRequestBody constructors for byte[] and Byt…

[cenedhryn]

…eBuffers (#3925)"

This reverts commit be45eb0.

Motivation and Context

Modifications

Testing

Screenshots (if appropriate)

Types of changes

• Bug fix (non-breaking change which fixes an issue)
• New feature (non-breaking change which adds functionality)

Checklist

• I have read the CONTRIBUTING document
• Local run of mvn install succeeds
• My code follows the code style of this project
• My change requires a change to the Javadoc documentation
• I have updated the Javadoc documentation accordingly
• I have added tests to cover my changes
• All new and existing tests passed
• I have added a changelog entry. Adding a new entry must be accomplished by running the scripts/new-change script and following the instructions. Commit the new file created by the script in .changes/next-release with your changes.
• My change is to implement 1.11 parity feature and I have updated LaunchChangelog

License

• I confirm that this pull request can be released under the Apache 2 license

[StephenFlavin]

Hi @cenedhryn,
Would it be possible to know what the motivation for the revert is?
it would re-introduce issue #1735 and conflict with my pr:#4096

[cenedhryn]

@StephenFlavin we got a user report where they ran into issues and were linking it to this code, but they have yet to provide a reliable repro case. This PR was created and sitting on standby, but enough days have passed that I think I can close it for now.

[StephenFlavin]

Ah, thanks @cenedhryn, happy to help debug if there's an issue in my changes