Bug #26249
closedTechnique can be created with technique ID starting with a forbidden character
Description
If creating a technique with a name starting with a character that will be escaped for generating the technique ID :
- example name : "/etc/hosts Compliance"
Rudder will auto-generate a Technique ID : “_etc_hosts_compliance”
- While displaying a message stating that the 1st character should be alphameric
- Where one cannot edit the Technique ID field as it is generated, but not an entry field.
If the the user goes to another technique in the techniques tree, then back to the usaved draft, the error message is gone and is becomes possible to save the technique with the “invalid” ID.
- And this technique seems to be usable and to work normally.
Files
Updated by Nicolas CHARLES 3 months ago
- Assignee set to Clark ANDRIANASOLO
- Priority changed from To review to 1 (highest)
Ok, there seems to be an error with draft and error management
Updated by Clark ANDRIANASOLO 3 months ago
- Status changed from New to In progress
Updated by Clark ANDRIANASOLO 3 months ago
- Status changed from In progress to Pending technical review
- Assignee changed from Clark ANDRIANASOLO to Vincent MEMBRÉ
- Pull Request set to https://github.com/Normation/rudder/pull/6154
Updated by Clark ANDRIANASOLO 3 months ago
I am able to reproduce this, the problem also occurs on the name validation (which is subject to additional API validation, so there is no actual problem when saving it) :
Updated by Clark ANDRIANASOLO 2 months ago
- Status changed from Pending technical review to Pending release
Applied in changeset rudder|157536f3e98f4e191eafd8f849932f16edfbe990.
Updated by Elaad FURREEDAN about 2 months ago
I'm able to save a technique with ID prefixed by _
like said in the GitHub ticket. I'm validating this
Updated by Elaad FURREEDAN about 2 months ago
- Fix check changed from To do to Checked
Updated by Vincent MEMBRÉ about 2 months ago
- Status changed from Pending release to Released
This bug has been fixed in Rudder 8.2.5 which was released today.