Project

General

Profile

Actions

Bug #13813

closed

While adding parameter to any host if the parameter value contain the multiple space , it's generating the faulty yaml

Added by Bryan Kearney about 8 years ago. Updated over 5 years ago.

Status:
Closed
Priority:
Normal
Assignee:
Category:
External Nodes
Target version:
Difficulty:
Triaged:
Fixed in Releases:
Found in Releases:

Description

Cloned from https://bugzilla.redhat.com/show_bug.cgi?id=1306395
Description of problem:
While adding parameter to any host if the parameter value contain the multiple space , it's generating the faulty yaml and hence the corresponding faulty change are reflected to the puppet agent.

Version-Release number of selected component (if applicable):
6.1

How reproducible:
Every time

Steps to Reproduce:
While adding parameter ( Host --> parameter --> add parameter ) to any host if the parameter value contain the multiple lines with spaces ( "\" shows up every 2nd entry in between parts of the line) , it's generating the faulty yaml add and hence the corresponding faulty change are reflected to the client config file.

extend    =    auth                          include              password-auth
             account     include                  password-auth

Corresponding yaml generated

extend: ! "auth               include              password-auth\r\naccount
    \        include               password-auth"                               

extra '\' added.

Actual results:

Expected results:

Additional info:

Actions #1

Updated by Dominic Cleal about 8 years ago

  • Category set to External Nodes
  • Status changed from New to Need more information
  • Assignee deleted (Ori Rabin)
  • Priority changed from High to Normal

A slash before a space is valid in YAML as far as I know, it shouldn't appear when loaded. The supplied YAML loads fine for me:

{"extend"=>"auth               include              password-auth\r\naccount         include               password-auth"}
Actions #2

Updated by Dominic Cleal about 8 years ago

  • Description updated (diff)
Actions #3

Updated by The Foreman Bot over 7 years ago

  • Status changed from Need more information to Ready For Testing
  • Assignee set to Shimon Shtein
  • Pull request https://github.com/theforeman/foreman/pull/3710 added
Actions #4

Updated by Shimon Shtein over 7 years ago

  • Status changed from Ready For Testing to Closed
  • % Done changed from 0 to 100
Actions #5

Updated by Ivan Necas over 7 years ago

  • Target version set to 1.7.0
Actions #6

Updated by Dominic Cleal over 7 years ago

  • translation missing: en.field_release set to 175
Actions

Also available in: Atom PDF