User tests: Successful: Unsuccessful:
Pull Request for Issue # .
Follow-up to PR #26375 .
With PR #26375 the possibility to use encrypted database connections was added to the CMS, and with PR #26888 it would be added to the installation process, too, if that is accepted.
Again thanks to @andrepereiradasilva for the implementation of these new features.
Because database connections default to socket connections in case if the database server is 'localhost' (which is of course more performant that a TCP/IP connection like e.g. to '127.0.0.1' or '::1'), and encryption is not possible with socket connections, this PR here changes the showon condition for the related fields in the server section of the Global Configuration form so that they are hidden when database server is 'localhost', and it adds an onchange JS to clear the fields when changing database server from something else to 'localhost'.
The JS added by this PR has the advantage that it can be used in future for other onchange events, too, e.g. if we want to clear the FTP related fields when disabling the FTP layer.
npm run build:js
.When the database host (field "Host") is "localhost", fields related to database connection encryption are shown depending on the value selected in field "Connection Envrytion". If "Two-way encryption" is selected, you can enter values in the text fields shown in this case.
See PR #26375 .
Status | New | ⇒ | Pending |
Category | ⇒ | Administration com_config |
Title |
|
Labels |
Added:
?
|
Category | Administration com_config | ⇒ | Administration com_config JavaScript Repository NPM Change |
Title |
|
Labels |
Added:
NPM Resource Changed
|
I have tested this item
I have tested this item
Status | Pending | ⇒ | Ready to Commit |
RTC.
Status | Ready to Commit | ⇒ | Fixed in Code Base |
Closed_Date | 0000-00-00 00:00:00 | ⇒ | 2019-11-28 22:30:35 |
Closed_By | ⇒ | HLeithner | |
Labels |
Added:
?
|
Thanks.
Potential testers please wait with testing until title does not contain "[WiP]" anymore and status on GitHub is not draft anymore.