Project

General

Profile

Bug #8261

Warn when the string length reaches the maximum value supported by the agent

Added by Alexis MOUSSET almost 5 years ago. Updated over 4 years ago.

Status:
Released
Priority:
N/A
Category:
Technique editor - UI/UX
Target version:
Severity:
User visibility:
Effort required:
Priority:

Description

The builder should display a warning and/or limit the field lengths according to the limits in the agent.


Subtasks

User story #8323: Limit the length of technique namesReleasedVincent MEMBRÉ2016-06-06Actions
User story #8465: Limit length of technique names to 100 chars in the ncf apiReleasedFrançois ARMAND2016-06-06Actions
#1

Updated by Alexis MOUSSET almost 5 years ago

  • Subject changed from Warn when the string length reacheqs the maximum value supported by the agent to Warn when the string length reaches the maximum value supported by the agent
#2

Updated by Jonathan CLARKE almost 5 years ago

  • Assignee set to Raphael GAUTHIER

Raphael, please add a check in ncf-builder to ensure that any fields with > 2000 characters cannot be extended, and display the following error message to the user: "Fields over 2000 characters are currently not supported. Please insert your content into a file, and copy it with a file_copy_* method, or use a template."

#3

Updated by Jonathan CLARKE almost 5 years ago

  • Project changed from Rudder to ncf
  • Category deleted (Web - Technique editor)
  • Target version set to 0.x
#4

Updated by Raphael GAUTHIER almost 5 years ago

  • Status changed from New to In progress
#5

Updated by Alexis MOUSSET almost 5 years ago

Classes conditions are also cut at 2048 when parsed (and hit the concat() max argument length at 4096 when writting a longer one).

#6

Updated by Alexis MOUSSET almost 5 years ago

Technique names should be kept quite short, because the name is used in the filename, which should be compatible with all filesystems.

For most systems the limit is around 255, and we need room for the extension. It may be a migration issue though, because some people may use names that are around the limit on some filesystems.

We should not break their installation, as it is not possible to change the same that has be given at creation, opening another ticket for this issue.

#7

Updated by Raphael GAUTHIER almost 5 years ago

  • Status changed from In progress to Pending technical review
  • Assignee changed from Raphael GAUTHIER to Vincent MEMBRÉ
  • Pull Request set to https://github.com/Normation/ncf/pull/360
#8

Updated by Anonymous almost 5 years ago

  • Status changed from Pending technical review to Pending release
  • % Done changed from 0 to 100
#9

Updated by Vincent MEMBRÉ over 4 years ago

  • Category set to Technique editor - UI/UX
#10

Updated by Vincent MEMBRÉ over 4 years ago

  • Status changed from Pending release to Released

This bug has been fixed in ncf version bundled with Rudder 2.11.24, 3.1.13 and 3.2.6 which were released today.

Also available in: Atom PDF