| Anonymous | Login | 2024-04-26 23:05 MDT |
| Main | My View | View Issues | Change Log | Roadmap | Repositories |
| View Issue Details [ Jump to Notes ] | [ Issue History ] [ Print ] | ||||||||
| ID | Project | Category | View Status | Date Submitted | Last Update | ||||
| 0022161 | ClearOS | app-certificate-manager - Certificate Manager | public | 2018-11-01 09:18 | 2020-05-04 05:29 | ||||
| Reporter | NickH | ||||||||
| Assigned To | |||||||||
| Priority | normal | Severity | tweak | Reproducibility | always | ||||
| Status | closed | Resolution | fixed | ||||||
| Platform | OS | OS Version | |||||||
| Product Version | 7.5.0 | ||||||||
| Target Version | 7.6.0 Updates | Fixed in Version | 7.6.0 | ||||||
| Summary | 0022161: When importing, the filter for allowable Names is incorrect | ||||||||
| Description | When you import a certificate, you have to give it a name but the restrictions on the allowable names has been incorrectly implemented. The filter in line 747 of /usr/clearos/apps/certificate_manager/libraries/External_Certificates.php is currently: return strlen($name) > 3 && preg_match("%^[a-zA-Z0-9\\-_]+(\\.[a-zA-Z0-9\\-_])*$%", $name); This will allow "test", "test.1", "test.1.2.3" but not "test.123". If you have a period in the name you must have only one character after it before having another period. It looks like the test string is incorrect and needs a "+" after the last "]" to read: return strlen($name) > 3 && preg_match("%^[a-zA-Z0-9\\-_]+(\\.[a-zA-Z0-9\\-_]+)*$%", $name); Then it will insist on at least one character after the period, but allow as many as you want. This will allow you to input an FQDN as a name but will not allow a name ending in a period. | ||||||||
| Tags | No tags attached. | ||||||||
| Attached Files | |||||||||
 Relationships |
|
 Relationships |
|
Notes |
|
|
(0008571) user2 2018-11-09 09:51 |
More restrictions were lifted: alphanumeric, dash, underscore and periods are fine: preg_match("/^[a-zA-Z0-9\-_\.]+$/", $name) |
|
(0014291) NickH (developer) 2020-05-04 05:29 |
Migrated to https://gitlab.com/clearos/clearfoundation/app-certificate-manager/-/issues/35 [^] |
|
Notes |
|
Issue History |
|||
| Date Modified | Username | Field | Change |
| 2018-11-01 09:18 | NickH | New Issue | |
| 2018-11-02 08:21 | user2 | Status | new => confirmed |
| 2018-11-02 08:21 | user2 | Product Version | 7.5.0 Updates => 7.5.0 |
| 2018-11-02 08:21 | user2 | Target Version | => 7.6.0 Updates |
| 2018-11-09 09:51 | user2 | Note Added: 0008571 | |
| 2018-11-09 09:51 | user2 | Status | confirmed => resolved |
| 2018-11-09 09:51 | user2 | Fixed in Version | => 7.6.0 |
| 2018-11-09 09:51 | user2 | Resolution | open => fixed |
| 2018-11-09 09:51 | user2 | Assigned To | => user2 |
| 2020-05-04 05:29 | NickH | Note Added: 0014291 | |
| 2020-05-04 05:29 | NickH | Status | resolved => closed |
| 2020-05-04 05:29 | NickH | Assigned To | user2 => |
|
Issue History |