diogo/symfony
Archived
1
0
Fork 0
Code Issues Pull Requests Packages Projects Releases Wiki Activity

[Notifier] Simplify OVH implementation

Browse Source
This commit is contained in:
Fabien Potencier 2020-02-12 17:24:01 +01:00
parent 5150fc51f6
commit 39f9ac2620
2 changed files with 3 additions and 12 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

13
src/Symfony/Component/Notifier/Bridge/OvhCloud/OvhCloudTransport.php
View File

@ -26,9 +26,7 @@ use Symfony\Contracts\HttpClient\HttpClientInterface;
*/ */
final class OvhCloudTransport extends AbstractTransport final class OvhCloudTransport extends AbstractTransport
{ {
private $endpoints = [ protected const HOST = 'eu.api.ovh.com';
'ovh-eu' => 'https://eu.api.ovh.com/1.0',
];
private $applicationKey; private $applicationKey;
private $applicationSecret; private $applicationSecret;
@ -45,13 +43,6 @@ final class OvhCloudTransport extends AbstractTransport
parent::__construct($client, $dispatcher); parent::__construct($client, $dispatcher);
} }
public function setEndpointName(?string $endpoint): self
{
$this->host = $this->endpoints[$endpoint] ?: self::HOST;
return $this;
}
public function __toString(): string public function __toString(): string
{ {
return sprintf('ovhcloud://%s?consumer_key=%s&service_name=%s', $this->getEndpoint(), $this->consumerKey, $this->serviceName); return sprintf('ovhcloud://%s?consumer_key=%s&service_name=%s', $this->getEndpoint(), $this->consumerKey, $this->serviceName);
@ -68,7 +59,7 @@ final class OvhCloudTransport extends AbstractTransport
throw new LogicException(sprintf('The "%s" transport only supports instances of "%s" (instance of "%s" given).', __CLASS__, SmsMessage::class, \get_class($message))); throw new LogicException(sprintf('The "%s" transport only supports instances of "%s" (instance of "%s" given).', __CLASS__, SmsMessage::class, \get_class($message)));
} }
$endpoint = sprintf('%s/sms/%s/jobs', $this->getEndpoint(), $this->serviceName); $endpoint = sprintf('https://%s/1.0/sms/%s/jobs', $this->getEndpoint(), $this->serviceName);
$content = [ $content = [
'charset' => 'UTF-8', 'charset' => 'UTF-8',

2
src/Symfony/Component/Notifier/Bridge/OvhCloud/OvhCloudTransportFactory.php
View File

@ -34,7 +34,7 @@ final class OvhCloudTransportFactory extends AbstractTransportFactory
$port = $dsn->getPort(); $port = $dsn->getPort();
if ('ovhcloud' === $scheme) { if ('ovhcloud' === $scheme) {
return (new OvhCloudTransport($applicationKey, $applicationSecret, $consumerKey, $serviceName, $this->client, $this->dispatcher))->setEndpointName($host)->setPort($port); return (new OvhCloudTransport($applicationKey, $applicationSecret, $consumerKey, $serviceName, $this->client, $this->dispatcher))->setHost($host)->setPort($port);
} }
throw new UnsupportedSchemeException($dsn, 'ovhcloud', $this->getSupportedSchemes()); throw new UnsupportedSchemeException($dsn, 'ovhcloud', $this->getSupportedSchemes());