Project

General

Profile

Actions

Bug #20133

closed

Technique editor allows id with only case difference, do an error, but still save technique

Added by François ARMAND about 3 years ago. Updated almost 3 years ago.

Status:
Released
Priority:
N/A
Category:
Web - Technique editor
Target version:
Severity:
UX impact:
User visibility:
Effort required:
Priority:
0
Name check:
To do
Fix check:
To do
Regression:

Description

So, I have a technique starting by an upper case. It's id also has an upper case (should not ?).
If I clone it, change first letter to lower case, try to save, I have an error which is totally not actionnable and should be change (something like "an error occured server side, try latter").
Then, in logs I get:

[2021-10-15 10:06:49+0000] ERROR com.normation.rudder.rest.RestUtils - "Could not create ncf technique <- Technique name and ID must be unique. 'technique with blocks' already used" 

As I'm still on the technique, I try to delete it.
There, I get a screen with 0 techniques remaining (I had only the other one, so perhaps it removed both based on the id, case insensitive?).
A bit of stress, then I click back on "techniques" in the main menu, and I get not only two techniques... But also two drafts ???

And yes, in the file system, contrary to what rudder and log told, there is a bunch of technique:

  • id must be case insensitive
  • error must be actionnable. What fail? How can I fix it?
  • if log say it was an error, there should not be files in the fs
  • draft must be deleted once technique is saved without error (what happened for the first one). I don't want draft for saved techniques.

Files


Related issues 2 (0 open2 closed)

Related to Rudder - Bug #20137: Technique draft can't be deletedReleasedFrançois ARMANDActions
Related to Rudder - Bug #20491: Error message in technique editor doesn't helpReleasedFrançois ARMANDActions
Actions #1

Updated by François ARMAND about 3 years ago

  • Related to Bug #20137: Technique draft can't be deleted added
Actions #2

Updated by Vincent MEMBRÉ about 3 years ago

  • Target version changed from 7.0.0~beta2 to 7.0.0~beta3
Actions #3

Updated by Vincent MEMBRÉ about 3 years ago

  • Target version changed from 7.0.0~beta3 to 7.0.0~rc1
Actions #4

Updated by Vincent MEMBRÉ about 3 years ago

  • Target version changed from 7.0.0~rc1 to 7.0.0~rc2
Actions #5

Updated by Vincent MEMBRÉ almost 3 years ago

  • Target version changed from 7.0.0~rc2 to 7.0.0~rc3
Actions #6

Updated by Benoît PECCATTE almost 3 years ago

  • Category set to Web - Technique editor
Actions #7

Updated by Elaad FURREEDAN almost 3 years ago

  • Status changed from New to In progress
  • Assignee set to Elaad FURREEDAN
Actions #8

Updated by Elaad FURREEDAN almost 3 years ago

  • Related to Bug #20491: Error message in technique editor doesn't help added
Actions #9

Updated by Elaad FURREEDAN almost 3 years ago

  • Status changed from In progress to Pending technical review
  • Assignee changed from Elaad FURREEDAN to Nicolas CHARLES
  • Pull Request set to https://github.com/Normation/rudder/pull/4079
Actions #10

Updated by Anonymous almost 3 years ago

  • Status changed from Pending technical review to Pending release
Actions #11

Updated by Vincent MEMBRÉ almost 3 years ago

  • Status changed from Pending release to Released

This bug has been fixed in Rudder 7.0.0~rc3 which was released today.

Actions

Also available in: Atom PDF