wildsideuk/laravel-mailguneu

Allow customising the Mailgun server URL to use EU servers.

laravel-mailguneu's Issues

Wrong URL with latest version of laravel 5.5

I did a composer update earlier today on my server and suddenly no mails were being sent. No errors, no logs... and no mail. Locally everything worked until I did the composer update. I'm not sure which package is at fault.

In any case, I did some digging into the MailgunTransporter and I saw that there is now an endpoint-parameter in the construct-function. This is used to build the url for the request.

Because of this, this package (WildSideUK/Laravel-MailgunEu), doesn't work any longer with the latest version of laravel 5.5 (at least 5.5.45, perhaps earlier).

Good news is, you can add an 'endpoint'-paramter in your services-config with the value eu.api.mailgun.net and everything will work out of the box.

PS: This is an already resolved issue (not really an issue at all) - but I didn't know where else to post it. I just wanted to spare my fellow humans the debug-hell I went through. Maybe put it in the readme?

Readme is inconsistent with implementation

The readme instructs users to provide a full url in the domain setting, but the actual implementation is hard-coding the EU endoint prefix to the domain value.

If the intent of this package is simply to allow drop-in replacement of the API endpoint, I suggest rewording the readme to be clear that this is all it does and doesn't offer any further configuration options, nor does it require the config values to be changed.