fix: only encrypt instance email settings if there are any

This commit is contained in:
peaklabs-dev
2024-12-23 17:41:38 +01:00
parent d7a0794bd9
commit 0a851ec3f2

View File

@@ -21,18 +21,20 @@ return new class extends Migration
$table->text('smtp_username')->nullable()->change(); $table->text('smtp_username')->nullable()->change();
}); });
$settings = DB::table('instance_settings')->get(); if (DB::table('instance_settings')->exists()) {
foreach ($settings as $setting) { $settings = DB::table('instance_settings')->get();
try { foreach ($settings as $setting) {
DB::table('instance_settings')->where('id', $setting->id)->update([ try {
'smtp_from_address' => $setting->smtp_from_address ? Crypt::encryptString($setting->smtp_from_address) : null, DB::table('instance_settings')->where('id', $setting->id)->update([
'smtp_from_name' => $setting->smtp_from_name ? Crypt::encryptString($setting->smtp_from_name) : null, 'smtp_from_address' => $setting->smtp_from_address ? Crypt::encryptString($setting->smtp_from_address) : null,
'smtp_recipients' => $setting->smtp_recipients ? Crypt::encryptString($setting->smtp_recipients) : null, 'smtp_from_name' => $setting->smtp_from_name ? Crypt::encryptString($setting->smtp_from_name) : null,
'smtp_host' => $setting->smtp_host ? Crypt::encryptString($setting->smtp_host) : null, 'smtp_recipients' => $setting->smtp_recipients ? Crypt::encryptString($setting->smtp_recipients) : null,
'smtp_username' => $setting->smtp_username ? Crypt::encryptString($setting->smtp_username) : null, 'smtp_host' => $setting->smtp_host ? Crypt::encryptString($setting->smtp_host) : null,
]); 'smtp_username' => $setting->smtp_username ? Crypt::encryptString($setting->smtp_username) : null,
} catch (Exception $e) { ]);
\Log::error('Error encrypting instance settings email columns: '.$e->getMessage()); } catch (Exception $e) {
\Log::error('Error encrypting instance settings email columns: '.$e->getMessage());
}
} }
} }
} }
@@ -43,11 +45,11 @@ return new class extends Migration
public function down(): void public function down(): void
{ {
Schema::table('instance_settings', function (Blueprint $table) { Schema::table('instance_settings', function (Blueprint $table) {
$table->text('smtp_from_address')->nullable()->change(); $table->string('smtp_from_address')->nullable()->change();
$table->text('smtp_from_name')->nullable()->change(); $table->string('smtp_from_name')->nullable()->change();
$table->text('smtp_recipients')->nullable()->change(); $table->string('smtp_recipients')->nullable()->change();
$table->text('smtp_host')->nullable()->change(); $table->string('smtp_host')->nullable()->change();
$table->text('smtp_username')->nullable()->change(); $table->string('smtp_username')->nullable()->change();
}); });
if (DB::table('instance_settings')->exists()) { if (DB::table('instance_settings')->exists()) {