Disable Expect 100-Continue by default in Apache 5 HTTP client

[alextwoods]

Disable Expect 100-Continue by default in Apache 5 HTTP client

Motivation and Context

Switch the default expectContinue from enabled to disabled in the new Apache 5 HTTP client. The Apache5 HTTP client applies Expect 100-Continue to all PUT requests with bodies. For non streaming cases (or for small streaming uploads) this is inefficient and results in multiple round trips.

S3 PutObject and UploadPart will still get expect-100 continue through the S3 Client.

Modifications

• Change default expectContinue from true to false.

Testing

Unit tests + manual testing.

Screenshots (if appropriate)

Types of changes

• Bug fix (non-breaking change which fixes an issue)
• [ x 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.

License

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

[sonarqubecloud]

Quality Gate passed

Issues
0 New issues
0 Accepted issues

Measures
0 Security Hotspots
100.0% Coverage on New Code
28.6% Duplication on New Code

See analysis details on SonarQube Cloud

[github-actions]

This pull request has been closed and the conversation has been locked. Comments on closed PRs are hard for our team to see. If you need more assistance, please open a new issue that references this one.