Merge pull request #23 from Muffinman/master

Adding subjectAlternativeNames support

README.md

@@ -111,6 +111,17 @@ $certificate->delete();
 $certificate->forceDelete();
 ```
 
+
+## Subject Alternative Names
+
+It's possible to specify Subject Alternative Names as below:
+
+```php
+LetsEncrypt::certificate('mydomain.com')
+        ->setSubjectAlternativeNames(['mydomain2.com'])
+        ->create(); 
+```
+
 ## Failure events
 
 If one of the jobs fails, one of the following events will be dispatched:

database/migrations/add_lets_encrypt_certificates_subject_alternative_names.php.stub

@@ -0,0 +1,22 @@
+<?php
+
+use Illuminate\Database\Migrations\Migration;
+use Illuminate\Database\Schema\Blueprint;
+use Illuminate\Support\Facades\Schema;
+
+class AddLetsEncryptCertificatesSubjectAlternativeNames extends Migration
+{
+    public function up()
+    {
+        Schema::table('lets_encrypt_certificates', function (Blueprint $table) {
+            $table->json('subject_alternative_names')->nullable()->after('domain');
+        });
+    }
+
+    public function down()
+    {
+        Schema::table('lets_encrypt_certificates', function (Blueprint $table) {
+            $table->dropColumn('subject_alternative_names');
+        });
+    }
+}

src/Jobs/RequestCertificate.php

@@ -38,7 +38,7 @@ public function __construct(LetsEncryptCertificate $certificate, int $tries = nu
 
     public function handle()
     {
-        $distinguishedName = new DistinguishedName($this->certificate->domain);
+        $distinguishedName = new DistinguishedName($this->certificate->domain, null, null, null, null, null, null, $this->certificate->subject_alternative_names);
         $csr = new CertificateRequest($distinguishedName, (new KeyPairGenerator())->generateKeyPair());
         $client = LetsEncrypt::createClient();
         $certificateResponse = $client->requestCertificate($this->certificate->domain, $csr);

src/Models/LetsEncryptCertificate.php

@@ -42,6 +42,7 @@ class LetsEncryptCertificate extends Model
 
     protected $casts = [
         'created' => 'boolean',
+        'subject_alternative_names' => 'json',
     ];
 
     public function newEloquentBuilder($query): LetsEncryptCertificateBuilder

src/PendingCertificate.php

@@ -42,6 +42,11 @@ class PendingCertificate
      */
     protected $delay = 0;
 
+    /**
+     * @var array
+     */
+    protected $subjectAlternativeNames = [];
+
     /**
      * PendingCertificate constructor.
      * @param string $domain
@@ -66,6 +71,7 @@ public function create(): LetsEncryptCertificate
 
         $certificate = LetsEncryptCertificate::create([
             'domain' => $this->domain,
+            'subject_alternative_names' => $this->subjectAlternativeNames,
         ]);
 
         RegisterAccount::withChain(array_merge([
@@ -117,6 +123,18 @@ public function renew(): LetsEncryptCertificate
         return $certificate;
     }
 
+
+    /**
+     * @param array $domains
+     * @return static
+     */
+    public function setSubjectAlternativeNames(array $domains): self
+    {
+        $this->subjectAlternativeNames = $domains;
+
+        return $this;
+    }
+
     /**
      * @param int $tries
      * @return static

tests/Facades/LetsEncryptTest.php

@@ -75,4 +75,19 @@ public function test_can_create_pending()
             RequestCertificate::class,
         ]);
     }
+
+    /** @test */
+    public function test_can_create_now_with_san()
+    {
+        Bus::fake();
+
+        $certificate = LetsEncrypt::certificate('somedomain.com')
+            ->setSubjectAlternativeNames(['other.somedomain.com'])
+            ->create();
+
+        $this->assertEquals('somedomain.com', $certificate->domain);
+        $this->assertEquals(['other.somedomain.com'], $certificate->subject_alternative_names);
+
+        Bus::assertDispatched(RegisterAccount::class);
+    }
 }

tests/TestCase.php

@@ -29,6 +29,8 @@ public function getEnvironmentSetUp($app)
         ]);
 
         include_once __DIR__.'/../database/migrations/create_lets_encrypt_certificates_table.php.stub';
+        include_once __DIR__.'/../database/migrations/add_lets_encrypt_certificates_subject_alternative_names.php.stub';
         (new \CreateLetsEncryptCertificatesTable())->up();
+        (new \AddLetsEncryptCertificatesSubjectAlternativeNames())->up();
     }
 }