Disable client side monitoring

christianc1 commented 4 years ago

Description of the Change

Disables Client Side Monitoring. Support for CSM Configuration was added to the AWS SDK in https://github.com/aws/aws-sdk-php/pull/1834. Recently our users have been getting the following errors when attempting to push snapshots:

Error Message: CSM 'port' value must be an integer!

This pull requests disables CSM as per the AWS SDK spec.

Alternate Designs

Benefits

Possible Drawbacks

I'm not exactly sure what Client Side Monitoring does, so disabling it might have unintended affects.

Verification Process

Testing in progress

Checklist:

[X] I have read the CONTRIBUTING document.
[x] My code follows the code style of this project.
[ ] My change requires a change to the documentation.
[ ] I have updated the documentation accordingly.
[ ] I have added tests to cover my change.
[ ] All new and existing tests passed.

Changelog Entry

Disables AWS Client Side Monitoring

christianc1 commented 4 years ago

CC: @ivanlopez @pypwalters

tlovett1 commented 4 years ago

I wasn't able to reproduce the problem but will merge anyway.

10up / wpsnapshots