Skip to content
Open
Show file tree
Hide file tree
Changes from all commits
Commits
File filter

Filter by extension

Filter by extension

Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
71 changes: 51 additions & 20 deletions src/Client.php
Original file line number Diff line number Diff line change
Expand Up @@ -116,6 +116,7 @@ class Client
* @param array $config
*
* @type string $mode The mode for ACME (production / staging)
* @type string $baseUri The base URI for the ACME API (cannot be supplied when mode is set)
* @type Filesystem $fs Filesystem for storage of static data
* @type string $basePath The base path for the filesystem (used to store account information and csr / keys
* @type string $username The acme username
Expand All @@ -125,6 +126,11 @@ class Client
public function __construct($config = [])
{
$this->config = $config;

if ($this->getOption('baseUri', false) && $this->getOption('mode', false)) {
throw new \LogicException('Both baseUri and mode cannot be supplied simultaneously.');
}

if ($this->getOption('fs', false)) {
$this->filesystem = $this->getOption('fs');
} else {
Expand All @@ -139,16 +145,21 @@ public function __construct($config = [])
}

/**
* Get an existing order by ID
* Get an existing order by ID or URL
*
* @param $id
* @param string $idOrUrl
* @return Order
* @throws \Exception
*/
public function getOrder($id): Order
public function getOrder($idOrUrl): Order
{
$url = str_replace('new-order', 'order', $this->getUrl(self::DIRECTORY_NEW_ORDER));
$url = $url . '/' . $this->getAccount()->getId() . '/' . $id;
if (strpos($idOrUrl, 'http') === 0) {
$url = $idOrUrl;
} else {
trigger_error("Warning: Constructing URL from ID. This may lead to unexpected behavior if the server uses a different base URL for existing orders.", E_USER_WARNING);
$url = str_replace('new-order', 'order', $this->getUrl(self::DIRECTORY_NEW_ORDER));
$url = $url . '/' . $this->getAccount()->getId() . '/' . $idOrUrl;
}
$response = $this->request($url, $this->signPayloadKid(null, $url));
$data = json_decode((string)$response->getBody(), true);

Expand Down Expand Up @@ -177,7 +188,7 @@ public function getOrder($id): Order
*/
public function isReady(Order $order): bool
{
$order = $this->getOrder($order->getId());
$order = $this->getOrder($order->getURL());
return $order->getStatus() == 'ready';
}

Expand Down Expand Up @@ -284,11 +295,9 @@ public function selfTest(Authorization $authorization, $type = self::VALIDATION_
*/
public function validate(Challenge $challenge, int $maxAttempts = 15): bool
{
$this->request(
$response = $this->request(
$challenge->getUrl(),
$this->signPayloadKid([
'keyAuthorization' => $challenge->getToken() . '.' . $this->getDigest()
], $challenge->getUrl())
$this->signPayloadKid([], $challenge->getUrl())
);

$data = [];
Expand All @@ -311,10 +320,11 @@ public function validate(Challenge $challenge, int $maxAttempts = 15): bool
* Return a certificate
*
* @param Order $order
* @param int $maxAttempts
* @return Certificate
* @throws \Exception
*/
public function getCertificate(Order $order): Certificate
public function getCertificate(Order $order, $maxAttempts = 15): Certificate
{
$privateKey = Helper::getNewKey($this->getOption('key_length', 4096));
$csr = Helper::getCsr($order->getDomains(), $privateKey);
Expand All @@ -328,7 +338,19 @@ public function getCertificate(Order $order): Certificate
)
);

$data = json_decode((string)$response->getBody(), true);
$data = [];
do {
$response = $this->request(
$order->getURL(),
$this->signPayloadKid(null, $order->getURL())
);
$data = json_decode((string)$response->getBody(), true);
if ($maxAttempts > 1 && $data['status'] != 'valid') {
sleep(ceil(15 / $maxAttempts));
}
$maxAttempts--;
} while ($maxAttempts > 0 && $data['status'] != 'valid');

$certificateResponse = $this->request(
$data['certificate'],
$this->signPayloadKid(null, $data['certificate'])
Expand Down Expand Up @@ -358,8 +380,15 @@ public function getAccount(): Account

$data = json_decode((string)$response->getBody(), true);
$accountURL = $response->getHeaderLine('Location');
$date = (new \DateTime())->setTimestamp(strtotime($data['createdAt']));
return new Account($data['contact'], $date, ($data['status'] == 'valid'), $data['initialIp'], $accountURL);

// Use the current date and time if 'createdAt' is not set
$createdAt = $data['createdAt'] ?? (new \DateTime())->format(\DateTime::ATOM);
$date = (new \DateTime())->setTimestamp(strtotime($createdAt));

// Use 'unknown' if 'initialIp' is not set
$initialIp = $data['initialIp'] ?? 'unknown';

return new Account($data['contact'], $date, ($data['status'] == 'valid'), $initialIp, $accountURL);
}

/**
Expand All @@ -370,9 +399,11 @@ protected function getHttpClient()
{
if ($this->httpClient === null) {
$config = [
'base_uri' => (
($this->getOption('mode', self::MODE_LIVE) == self::MODE_LIVE) ?
self::DIRECTORY_LIVE : self::DIRECTORY_STAGING),
'base_uri' => $this->getOption(
'baseUri',
($this->getOption('mode', self::MODE_LIVE) == self::MODE_LIVE) ?
self::DIRECTORY_LIVE : self::DIRECTORY_STAGING
),
];
if ($this->getOption('source_ip', false) !== false) {
$config['curl.options']['CURLOPT_INTERFACE'] = $this->getOption('source_ip');
Expand Down Expand Up @@ -479,7 +510,7 @@ protected function getSelfTestDNSClient()
protected function init()
{
//Load the directories from the LE api
$response = $this->getHttpClient()->get('/directory');
$response = $this->getHttpClient()->get('');
$result = \GuzzleHttp\json_decode((string)$response->getBody(), true);
$this->directories = $result;

Expand Down Expand Up @@ -738,7 +769,7 @@ protected function signPayloadJWK($payload, $url): array
*/
protected function signPayloadKid($payload, $url): array
{
$payload = is_array($payload) ? str_replace('\\/', '/', json_encode($payload)) : '';
$payload = is_array($payload) ? str_replace('\\/', '/', json_encode((object)$payload)) : '';
$payload = Helper::toSafeString($payload);
$protected = Helper::toSafeString(json_encode($this->getKID($url)));

Expand All @@ -753,4 +784,4 @@ protected function signPayloadKid($payload, $url): array
'signature' => Helper::toSafeString($signature),
];
}
}
}
9 changes: 9 additions & 0 deletions src/Data/Authorization.php
Original file line number Diff line number Diff line change
Expand Up @@ -79,6 +79,15 @@ public function getChallenges(): array
return $this->challenges;
}

/**
* Return this authorization digest
* @return string
*/
public function getDigest(): string
{
return $this->digest;
}

/**
* Return the HTTP challenge
* @return Challenge|bool
Expand Down