Improve error logging

Hi, I rerolled the patch up to 8.3.3 version.

I wasn't able to create an interdiff file because the previous patch doesn't apply anymore.

I've found out that In some cases the getAwsErrorMessage() method is returns an empty string. Like when we try to use an nonexistent bucket.
So I've added a fallback to it.

All the comments in the previous patch are now resolved but the one related with providing a fallback to the logger parameter. It mentioned that this would be needed because we were in beta, which doesn't seems to be the case anymore.

It mentioned that this would be needed because we were in beta, which doesn't seems to be the case anymore.

It’s a “once a BETA release is made, BC measure should be done” which especially applies to Stable since we didn’t mark the class internal it may be being extended.

The simple fallback for this is, when the parameter is null fallback to calling \Drupal::service() to get the logger and raise a deprecation notice.

Committed to 8.x-3.x needs port to 4.x