Feature/aws ses

[Orkin]

Add Amazon SES provider

[Orkin]

@danizord @bakura10 @Ocramius

[bakura10]

SES is a very old service and as a consequence I think we should not be as tight in the dependency. I think a slightly lower version should be enough.

[Orkin]

Here I need 3.36.25 because of templated email management

[bakura10]

Good point! Let's keep it this way then.

[bakura10]

Why changing that?

[Orkin]

Because I prefer like this but if you want I can remove it ;)

[bakura10]

Yes, this is not a useful change :p.

[bakura10]

I'd recommend you to make this package with an optional dependency to https://github.com/zf-fr/zfr-aws-utils

This is actually a super lightweight library that integrates with AWS, and allows you to set that kind of config:

return [
   'aws' => [
       // AWS config
   ]
]

and then allow to get a client:

$sdk = $container->get(Aws\Sdk::class);
$ses = $sdk->createSesClient();

This has the advantage to centralize all AWS config into a single config key and therefore avoid having it defined for each library that uses a client.

So I suggest:

• Add this library as a "suggested" library
• Update documentation to indicate that SES requires usage of this library
• Update doucmenttaion for the config

What do you think? :)

[Orkin]

I think it's good but ZfrAwsUtils has references to Zend Service Manager with ConfigAbstractFactory key ><. For now I just want to be PSR compliant.

[bakura10]

ZfrAwsUtils has no required dependency against Zend Service Manager: https://github.com/zf-fr/zfr-aws-utils/blob/master/composer.json#L22 and the factory to create the client is 100% PSR compliant so can be re-used with anything: https://github.com/zf-fr/zfr-aws-utils/blob/master/src/Container/SdkFactory.php :)

[Orkin]

Ok ok I made the change :p

[bakura10]

I didn't use it anywhere in the library so for consistency I'd suggest you get rid of this.

[bakura10]

Remove useless empty line.

[bakura10]

As it’s not needed for people using postmark I’d suggest you move this into the suggested dependencies. This will prevent pulling this and the aws sel for people not using it

[Orkin]

Oups my bad ;)

[bakura10]

Looks good. Last thing is : can you added it to the changelog (https://github.com/zf-fr/zfr-mail/blob/master/CHANGELOG.md) ? I'll tag this as 2.2.0.

[Orkin]

@bakura10 done 👍

[bakura10]

Thanks!!