fix & prevent errors on decrypt data

Author: lao9s

resources/js/Components/Post/PostPreviewProviders.vue

@@ -50,7 +50,13 @@ const previews = computed(() => {
 <template>
     <template v-if="selectedAccounts.length">
         <template v-for="preview in previews" :key="preview.id">
-            <Alert v-if="preview.account.errors && preview.account.errors.length" variant="error" :closeable="false" class="mb-1">{{ preview.account.errors.join(',') }}</Alert>
+            <Alert v-if="preview.account.errors && preview.account.errors.length"
+                   variant="error"
+                   :closeable="false"
+                   class="mb-1">
+                <span v-html="preview.account.errors.join('</br>')"></span>
+            </Alert>
+
             <div class="mb-lg last:mb-0">
                 <component :is="preview.providerComponent"
                            :name="preview.account.name"

src/Actions/AccountPublishPost.php

@@ -3,6 +3,7 @@
 namespace Inovector\Mixpost\Actions;
 
 use Illuminate\Database\Eloquent\Collection;
+use Illuminate\Support\Facades\App;
 use Inovector\Mixpost\Support\Log;
 use Inovector\Mixpost\Facades\SocialProviderManager;
 use Inovector\Mixpost\Models\Account;
@@ -25,9 +26,9 @@ public function __invoke(Account $account, Post $post): void
         $body = $this->cleanBody($content[0]['body']);
         $media = $this->collectMedia($content[0]['media']);
 
-        $provider = SocialProviderManager::connect($account->provider, $account->values())->useAccessToken($account->access_token->toArray());
-
         try {
+            $provider = SocialProviderManager::connect($account->provider, $account->values())->useAccessToken($account->access_token->toArray());
+
             $response = $provider->publishPost($body, $media);
 
             if (isset($response['errors'])) {
@@ -38,7 +39,7 @@ public function __invoke(Account $account, Post $post): void
                 $this->insertProviderPostId($post, $account, $response['id']);
             }
         } catch (Exception $exception) {
-            Log::error("Publish: {$exception->getMessage()}",
+            Log::error($exception->getMessage(),
                 [
                     'account_id' => $account->id,
                     'account_name' => $account->name,
@@ -50,6 +51,10 @@ public function __invoke(Account $account, Post $post): void
 
             $errors = ['Unexpected internal error.'];
 
+            if (!App::environment('production')) {
+                $errors[] = $exception->getMessage();
+            }
+
             $this->insertErrors($post, $account, $errors);
         }
     }

src/Casts/EncryptArrayObject.php

@@ -0,0 +1,28 @@
+<?php
+
+namespace Inovector\Mixpost\Casts;
+
+use Illuminate\Contracts\Database\Eloquent\CastsAttributes;
+use Illuminate\Database\Eloquent\Casts\ArrayObject;
+use Illuminate\Support\Facades\Crypt;
+
+class EncryptArrayObject implements CastsAttributes
+{
+    public function get($model, $key, $value, $attributes): ?ArrayObject
+    {
+        if (isset($attributes[$key])) {
+            return new ArrayObject(json_decode(Crypt::decryptString($attributes[$key]), true));
+        }
+
+        return null;
+    }
+
+    public function set($model, string $key, $value, array $attributes): ?array
+    {
+        if (!is_null($value)) {
+            return [$key => Crypt::encryptString(json_encode($value))];
+        }
+
+        return null;
+    }
+}

src/Models/Account.php

@@ -2,7 +2,7 @@
 
 namespace Inovector\Mixpost\Models;
 
-use Illuminate\Database\Eloquent\Casts\AsEncryptedArrayObject;
+use Inovector\Mixpost\Casts\EncryptArrayObject;
 use Illuminate\Database\Eloquent\Factories\HasFactory;
 use Illuminate\Database\Eloquent\Model;
 use Illuminate\Support\Arr;
@@ -28,7 +28,7 @@ class Account extends Model
     protected $casts = [
         'media' => AccountMediaCast::class,
         'data' => 'array',
-        'access_token' => AsEncryptedArrayObject::class
+        'access_token' => EncryptArrayObject::class
     ];
 
     protected $hidden = [

src/Models/Service.php

@@ -2,7 +2,7 @@
 
 namespace Inovector\Mixpost\Models;
 
-use Illuminate\Database\Eloquent\Casts\AsEncryptedArrayObject;
+use Inovector\Mixpost\Casts\EncryptArrayObject;
 use Illuminate\Database\Eloquent\Factories\HasFactory;
 use Illuminate\Database\Eloquent\Model;
 use Inovector\Mixpost\Facades\Services as ServicesFacade;
@@ -19,7 +19,7 @@ class Service extends Model
     ];
 
     protected $casts = [
-        'credentials' => AsEncryptedArrayObject::class
+        'credentials' => EncryptArrayObject::class
     ];
 
     protected $hidden = [

src/Services.php

@@ -60,19 +60,21 @@ public function put(string $name, array $value): void
 
     public function get(string $name, null|string $credentialKey = null)
     {
-        $value = $this->getFromCache($name, function () use ($name) {
+        $defaultPayload = Arr::get($this->form(), $name, []);
+
+        $value = $this->getFromCache($name, function () use ($name, $defaultPayload) {
             $dbRecord = Service::where('name', $name)->first();
 
             try {
-                $payload = $dbRecord ? $dbRecord->credentials->toArray() : Arr::get($this->form(), $name, []);
+                $payload = $dbRecord ? $dbRecord->credentials->toArray() : $defaultPayload;
 
                 $this->put($name, $payload);
 
                 return $payload;
             } catch (DecryptException $exception) {
                 $this->logDecryptionError($name, $exception);
 
-                return [];
+                return $defaultPayload;
             }
         });
 
@@ -82,7 +84,7 @@ public function get(string $name, null|string $credentialKey = null)
             } catch (DecryptException $exception) {
                 $this->logDecryptionError($name, $exception);
 
-                $value = [];
+                $value = $defaultPayload;
             }
         }