Update dependencies

.travis.yml

@@ -11,10 +11,12 @@ cache:
     - vendor
 
 install:
-  - phpenv config-rm xdebug.ini || true
+  - phpenv config-rm xdebug.ini
+  - composer self-update
   - composer config --global discard-changes true
-  - composer update
-  - composer require satooshi/php-coveralls dev-master --dev
+  - if [[ ${TRAVIS_PHP_VERSION:0:3} == "5.5" ]]; then composer require --dev --no-update phpunit/phpunit ^4; fi
+  - composer require satooshi/php-coveralls dev-master --dev --no-update
+  - composer update --ignore-platform-reqs
   - composer show --installed
 
 script:

bin/acme

@@ -135,7 +135,6 @@ try {
 $injector = new Injector;
 $injector->share($climate);
 $injector->share(new AcmeFactory);
-$injector->share(new Amp\Artax\Client(new Amp\Artax\Cookie\NullCookieJar));
 
 $command = $injector->make($class);
 

composer.json

@@ -22,9 +22,9 @@
         "symfony/yaml": "^3.0"
     },
     "require-dev": {
-        "phpunit/phpunit": "^4|^5",
-        "friendsofphp/php-cs-fixer": "^1.9",
-        "macfja/phar-builder": "^0.2.5"
+        "phpunit/phpunit": "^5",
+        "fabpot/php-cs-fixer": "^1.9",
+        "macfja/phar-builder": "dev-events-dev-files"
     },
     "license": "MIT",
     "authors": [

doc/installation.md

@@ -2,7 +2,7 @@
 
 ## Installation using Phar
 
-This is the preferred installation method for usage on a production system. You can download `acme-client.phar` in the [release section](https://github.com/kelunik/acme-client/releases).
+This is the preferred installation method for usage on a production system.
 
 ### Requirements
 

doc/usage.md

@@ -77,25 +77,8 @@ the script will be quiet to be cron friendly. If an error occurs, the script wil
 You should execute `acme-client auto` as a daily cron. It's recommended to setup e-mail notifications for all output of
 that script.
 
-Create a new script, e.g. in `/usr/local/bin/acme-renew`. The `PATH` might need to be modified to suit your system. 
-
 ```bash
-#!/usr/bin/env bash
-
-export PATH='/usr/local/sbin:/usr/local/bin:/usr/sbin:/usr/bin:/sbin:/bin'
-
-acme-client auto
-
-RC=$?
-
-if [ $RC = 4 ] || [ $RC = 5 ]; then
-    service nginx reload
-fi
-```
-
-```sh
-# Cron Job Configuration
-0 0 * * * /usr/local/bin/acme-renew
+0 0 * * * acme-client auto; exit=$?; if [[ $exit = 4 ]] || [[ $exit = 5 ]]; then service nginx reload; fi
 ```
 
 | Exit Code | Description |

src/Commands/Auto.php

@@ -37,6 +37,8 @@ class Auto implements Command {
      * @return \Generator
      */
     private function doExecute(Manager $args) {
+        $server = $args->get("server");
+        $storage = $args->get("storage");
         $configPath = $args->get("config");
 
         try {
@@ -53,30 +55,6 @@ class Auto implements Command {
             return;
         }
 
-        if ($args->defined("server")) {
-            $config["server"] = $args->get("server");
-        } else if (!isset($config["server"]) && $args->exists("server")) {
-            $config["server"] = $args->get("server");
-        }
-
-        if ($args->defined("storage")) {
-            $config["storage"] = $args->get("storage");
-        } else if (!isset($config["storage"]) && $args->exists("storage")) {
-            $config["storage"] = $args->get("storage");
-        }
-
-        if (!isset($config["server"])) {
-            $this->climate->error("Config file ({$configPath}) didn't have a 'server' set nor was it passed as command line argument.");
-            yield new CoroutineResult(self::EXIT_CONFIG_ERROR);
-            return;
-        }
-
-        if (!isset($config["storage"])) {
-            $this->climate->error("Config file ({$configPath}) didn't have a 'storage' set nor was it passed as command line argument.");
-            yield new CoroutineResult(self::EXIT_CONFIG_ERROR);
-            return;
-        }
-
         if (!isset($config["email"])) {
             $this->climate->error("Config file ({$configPath}) didn't have a 'email' set.");
             yield new CoroutineResult(self::EXIT_CONFIG_ERROR);
@@ -89,22 +67,14 @@ class Auto implements Command {
             return;
         }
 
-        if (isset($config["challenge-concurrency"]) && !is_numeric($config["challenge-concurrency"])) {
-            $this->climate->error("Config file ({$configPath}) defines an invalid 'challenge-concurrency' value.");
-            yield new CoroutineResult(self::EXIT_CONFIG_ERROR);
-            return;
-        }
-
-        $concurrency = isset($config["challenge-concurrency"]) ? (int) $config["challenge-concurrency"] : null;
-
         $command = implode(" ", array_map("escapeshellarg", [
             PHP_BINARY,
             $GLOBALS["argv"][0],
             "setup",
             "--server",
-            $config["server"],
+            $server,
             "--storage",
-            $config["storage"],
+            $storage,
             "--email",
             $config["email"],
         ]));
@@ -121,16 +91,22 @@ class Auto implements Command {
             return;
         }
 
+        $certificateChunks = array_chunk($config["certificates"], 10, true);
+
         $errors = [];
         $values = [];
 
-        foreach ($config["certificates"] as $i => $certificate) {
-            try {
-                $result = (yield \Amp\resolve($this->checkAndIssue($certificate, $config["server"], $config["storage"], $concurrency)));
-                $values[$i] = $result;
-            } catch (\Exception $e) {
-                $errors[$i] = $e;
+        foreach ($certificateChunks as $certificateChunk) {
+            $promises = [];
+
+            foreach ($certificateChunk as $certificate) {
+                $promises[] = \Amp\resolve($this->checkAndIssue($certificate, $server, $storage));
             }
+
+            list($chunkErrors, $chunkValues) = (yield \Amp\any($promises));
+
+            $errors += $chunkErrors;
+            $values += $chunkValues;
         }
 
         $status = [
@@ -173,11 +149,10 @@ class Auto implements Command {
      * @param array  $certificate certificate configuration
      * @param string $server server to use for issuance
      * @param string $storage storage directory
-     * @param int|null $concurrency concurrent challenges
      * @return \Generator
      * @throws AcmeException if something does wrong
      */
-    private function checkAndIssue(array $certificate, $server, $storage, $concurrency = null) {
+    private function checkAndIssue(array $certificate, $server, $storage) {
         $domainPathMap = $this->toDomainPathMap($certificate["paths"]);
         $domains = array_keys($domainPathMap);
         $commonName = reset($domains);
@@ -192,8 +167,6 @@ class Auto implements Command {
             $storage,
             "--name",
             $commonName,
-            "--names",
-            implode(",", $domains),
         ];
 
         $command = implode(" ", array_map("escapeshellarg", $args));
@@ -233,11 +206,6 @@ class Auto implements Command {
                 $args[] = $certificate["bits"];
             }
 
-            if ($concurrency) {
-                $args[] = "--challenge-concurrency";
-                $args[] = $concurrency;
-            }
-
             $command = implode(" ", array_map("escapeshellarg", $args));
 
             $process = new Process($command);
@@ -309,15 +277,9 @@ MESSAGE;
     }
 
     public static function getDefinition() {
-        $server = \Kelunik\AcmeClient\getArgumentDescription("server");
-        $storage = \Kelunik\AcmeClient\getArgumentDescription("storage");
-
-        $server["required"] = false;
-        $storage["required"] = false;
-
         $args = [
-            "server" => $server,
-            "storage" => $storage,
+            "server" => \Kelunik\AcmeClient\getArgumentDescription("server"),
+            "storage" => \Kelunik\AcmeClient\getArgumentDescription("storage"),
             "config" => [
                 "prefix" => "c",
                 "longPrefix" => "config",

src/Commands/Check.php

@@ -45,18 +45,6 @@ class Check implements Command {
         $this->climate->br();
         $this->climate->whisper("    Certificate is valid until " . date("d.m.Y", $cert->getValidTo()))->br();
 
-        if ($args->defined("names")) {
-            $names = array_map("trim", explode(",", $args->get("names")));
-            $missingNames = array_diff($names, $cert->getNames());
-
-            if ($missingNames) {
-                $this->climate->comment("    The following names are not covered: " . implode(", ", $missingNames))->br();
-
-                yield new CoroutineResult(1);
-                return;
-            }
-        }
-
         if ($cert->getValidTo() > time() + $args->get("ttl") * 24 * 60 * 60) {
             yield new CoroutineResult(0);
             return;
@@ -82,11 +70,6 @@ class Check implements Command {
                 "defaultValue" => 30,
                 "castTo" => "int",
             ],
-            "names" => [
-                "longPrefix" => "names",
-                "description" => "Names that must be covered by the certificate identified based on the common name. Names have to be separated by commas.",
-                "required" => false,
-            ],
         ];
     }
 }

src/Commands/Issue.php

@@ -82,9 +82,7 @@ class Issue implements Command {
         $acme = $this->acmeFactory->build($server, $keyPair);
         $errors = [];
 
-        $concurrency = $args->get("challenge-concurrency");
-
-        $domainChunks = array_chunk($domains, \min(20, \max($concurrency, 1)), true);
+        $domainChunks = array_chunk($domains, 10, true);
 
         foreach ($domainChunks as $domainChunk) {
             $promises = [];
@@ -155,7 +153,7 @@ class Issue implements Command {
         $challengeStore = new ChallengeStore($path);
 
         try {
-            yield $challengeStore->put($token, $payload, isset($user) ? $user : null);
+            $challengeStore->put($token, $payload, isset($user) ? $user : null);
 
             yield $acme->verifyHttp01Challenge($domain, $token, $payload);
             yield $acme->answerChallenge($challenge->uri, $payload);
@@ -185,7 +183,7 @@ class Issue implements Command {
 
             foreach ($domainChunk as $domain) {
                 $promises[$domain] = \Amp\Dns\resolve($domain, [
-                    "types" => [Record::A, Record::AAAA],
+                    "types" => [Record::A],
                     "hosts" => false,
                 ]);
             }
@@ -196,13 +194,7 @@ class Issue implements Command {
         }
 
         if (!empty($errors)) {
-            $failedDomains = implode(", ", array_keys($errors));
-            $reasons = implode("\n\n", array_map(function ($exception) {
-                /** @var \Exception|\Throwable $exception */
-                return get_class($exception) . ": " . $exception->getMessage();
-            }, $errors));
-
-            throw new AcmeException("Couldn't resolve the following domains to an IPv4 nor IPv6 record: {$failedDomains}\n\n{$reasons}");
+            throw new AcmeException("Couldn't resolve the following domains to an IPv4 record: " . implode(", ", array_keys($errors)));
         }
     }
 
@@ -253,12 +245,6 @@ class Issue implements Command {
                 "defaultValue" => 2048,
                 "castTo" => "int",
             ],
-            "challenge-concurrency" => [
-                "longPrefix" => "challenge-concurrency",
-                "description" => "Number of challenges to be solved concurrently.",
-                "defaultValue" => 10,
-                "castTo" => "int",
-            ],
         ];
     }
 }

src/Commands/Setup.php

@@ -3,7 +3,6 @@
 namespace Kelunik\AcmeClient\Commands;
 
 use Amp\CoroutineResult;
-use Amp\Dns\NoRecordException;
 use Amp\Dns\Record;
 use Amp\Dns\ResolutionException;
 use InvalidArgumentException;
@@ -81,10 +80,8 @@ class Setup implements Command {
 
         try {
             yield \Amp\Dns\query($host, Record::MX);
-        } catch (NoRecordException $e) {
-            throw new AcmeException("No MX record defined for '{$host}'");
         } catch (ResolutionException $e) {
-            throw new AcmeException("Dns query for an MX record on '{$host}' failed for the following reason: " . $e->getMessage(), null, $e);
+            throw new AcmeException("No MX record defined for '{$host}'");
         }
     }