Bubble up revert data on failed Address.sendValue

[arr00]

Fixes #5184

PR Checklist

• Tests
• Documentation
• Changeset entry (run npx changeset add)

[changeset-bot]

🦋 Changeset detected

Latest commit: 6583803

The changes in this PR will be included in the next version bump.

This PR includes changesets to release 1 package

Name	Type
openzeppelin-solidity	Minor

Not sure what this means? Click here to learn what changesets are.

Click here if you're a maintainer who wants to add another changeset to this PR

[CodeSandwich]

Would the solution below be inferior? It seems that it should be exactly equivalent while deduplicating code.

function sendValue(address payable recipient, uint256 amount) internal {
    functionCallWithValue(address(recipient), "", amount);
}

[arr00]

Would the solution below be inferior? It seems that it should be exactly equivalent while deduplicating code.

function sendValue(address payable recipient, uint256 amount) internal {
    functionCallWithValue(address(recipient), "", amount);
}

functioncallWithValue in turn calls verifyCallResultFromTarget which doesn't allow calling EOAs.

[CodeSandwich]

Ahh, good point, I missed that detail 👍