smw/deskflow
1
0
Fork 0
Code Issues Pull requests Projects Releases Packages Wiki Activity Actions

fix: settings dialog can have tls options enabled when they should not be leading to a case where tls gen fails as it not enabled fixes 7883

Browse source
This commit is contained in:
sithlord48 2025-02-24 19:13:03 -05:00 committed by Chris Rizzitello
parent 5b54849fd6
commit 193c28af53
2 changed files with 72 additions and 64 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

18
src/apps/deskflow-gui/dialogs/SettingsDialog.cpp
View file

@ -240,6 +240,7 @@ void SettingsDialog::loadFromConfig()
else else
ui->rb_icon_mono->setChecked(true); ui->rb_icon_mono->setChecked(true);
qDebug() << "load from config done";
updateTlsControls(); updateTlsControls();
} }
@ -255,12 +256,18 @@ void SettingsDialog::updateTlsControls()
const auto tlsEnabled = m_tlsUtility.isEnabled(); const auto tlsEnabled = m_tlsUtility.isEnabled();
const auto writable = m_appConfig.isActiveScopeWritable(); const auto writable = m_appConfig.isActiveScopeWritable();
const auto enabled = writable && tlsEnabled;
ui->m_pLineEditTlsCertPath->setText(m_appConfig.tlsCertPath());
ui->chkRequireClientCert->setChecked(m_appConfig.requireClientCerts());
ui->m_pCheckBoxEnableTls->setChecked(tlsEnabled);
ui->m_pCheckBoxEnableTls->setEnabled(writable); ui->m_pCheckBoxEnableTls->setEnabled(writable);
ui->m_pCheckBoxEnableTls->setChecked(writable && tlsEnabled); ui->m_pComboBoxTlsKeyLength->setEnabled(enabled);
ui->m_pLineEditTlsCertPath->setText(m_appConfig.tlsCertPath()); ui->widgetTlsCert->setEnabled(enabled);
ui->chkRequireClientCert->setEnabled(writable); ui->m_pLabelTlsKeyLength->setEnabled(enabled);
ui->chkRequireClientCert->setChecked(m_appConfig.requireClientCerts()); ui->m_pPushButtonTlsRegenCert->setEnabled(enabled);
ui->chkRequireClientCert->setEnabled(enabled);
} }
void SettingsDialog::updateTlsControlsEnabled() void SettingsDialog::updateTlsControlsEnabled()
@ -273,8 +280,7 @@ void SettingsDialog::updateTlsControlsEnabled()
ui->m_pLabelTlsKeyLength->setEnabled(enabled); ui->m_pLabelTlsKeyLength->setEnabled(enabled);
ui->m_pComboBoxTlsKeyLength->setEnabled(enabled); ui->m_pComboBoxTlsKeyLength->setEnabled(enabled);
ui->m_pLabelTlsCert->setEnabled(enabled); ui->m_pLabelTlsCert->setEnabled(enabled);
ui->m_pLineEditTlsCertPath->setEnabled(enabled); ui->widgetTlsCert->setEnabled(enabled);
ui->m_pPushButtonTlsCertPath->setEnabled(enabled);
ui->m_pPushButtonTlsRegenCert->setEnabled(enabled); ui->m_pPushButtonTlsRegenCert->setEnabled(enabled);
ui->chkRequireClientCert->setEnabled(enabled); ui->chkRequireClientCert->setEnabled(enabled);
} }

6
src/apps/deskflow-gui/dialogs/SettingsDialog.ui
View file

@ -7,7 +7,7 @@
<x>0</x> <x>0</x>
<y>0</y> <y>0</y>
<width>550</width> <width>550</width>
<height>652</height> <height>684</height>
</rect> </rect>
</property> </property>
<property name="windowTitle"> <property name="windowTitle">
@ -275,9 +275,10 @@
</layout> </layout>
</item> </item>
<item> <item>
<widget class="QWidget" name="widgetTlsCert" native="true">
<layout class="QHBoxLayout" name="horizontalLayout_6"> <layout class="QHBoxLayout" name="horizontalLayout_6">
<property name="topMargin"> <property name="topMargin">
<number>0</number> <number>1</number>
</property> </property>
<item> <item>
<widget class="QLabel" name="m_pLabelTlsCert"> <widget class="QLabel" name="m_pLabelTlsCert">
@ -332,6 +333,7 @@
</widget> </widget>
</item> </item>
</layout> </layout>
</widget>
</item> </item>
<item> <item>
<layout class="QHBoxLayout" name="horizontalLayout_7"> <layout class="QHBoxLayout" name="horizontalLayout_7">