initial implementation of an S3 ingestor for MCP

[justinedelson]

includes some internal refactoring to avoid code duplication between S3 and File ingestor, although some still exists

[justinedelson]

Here's the problem with the StringUtils class... S3 Mock depends upon a method added in the 1.10.77 release of the AWS SDK. Naturally, AEM uses 1.10.76. So with just the AEM version on the classpath, S3 Mock fails miserably. So I took the latest copy of the StringUtils class and incorporated in into the codebase (as a test file) so that it would get picked up at test time. Would it have been better if S3 Mock used Commons StringUtils? sure. But I don't think that's likely to change.

[coveralls]

Coverage increased (+0.4%) to 29.756% when pulling e308f97 on feature/s3-asset-ingestor into 5e1694e on master.

[coveralls]

Coverage increased (+0.4%) to 29.775% when pulling 4073a97 on feature/s3-asset-ingestor into 5e1694e on master.

[coveralls]

Coverage increased (+0.4%) to 29.775% when pulling 68a0479 on feature/s3-asset-ingestor into 5e1694e on master.

[coveralls]

Coverage increased (+0.5%) to 29.895% when pulling ec43976 on feature/s3-asset-ingestor into 5e1694e on master.

[coveralls]

Coverage increased (+0.5%) to 29.89% when pulling b642a57 on feature/s3-asset-ingestor into 5e1694e on master.

[coveralls]

Coverage increased (+0.6%) to 29.979% when pulling 9fb2f2d on feature/s3-asset-ingestor into 5e1694e on master.

[coveralls]

Coverage increased (+0.8%) to 30.16% when pulling 5897f76 on feature/s3-asset-ingestor into 5e1694e on master.

[justinedelson]

@badvision @davidjgonzalez I think this is now ready to merge.

[badvision]

Looks good to me! 👍

[coveralls]

Coverage increased (+0.8%) to 30.16% when pulling 714e18a on feature/s3-asset-ingestor into 5e1694e on master.

[coveralls]

Coverage increased (+0.8%) to 30.159% when pulling 70bb46d on feature/s3-asset-ingestor into 5e1694e on master.