Project

General

Profile

Bug #13813

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 4 years ago. Updated over 1 year ago.

Status:
Closed
Priority:
Normal
Assignee:
Category:
External Nodes
Target version:
Difficulty:
Triaged:
Bugzilla link:
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:

Associated revisions

Revision 0069c98d (diff)
Added by Shimon Shtein over 3 years ago

Fixes #13813 - Removed "\ " so ruby 1.8 could read the ENC

Revision e3a40b55 (diff)
Added by Shimon Shtein over 3 years ago

Fixes #13813 - Removed "\ " so ruby 1.8 could read the ENC

(cherry picked from commit 0069c98dcb50a8960f92397921cfb8a98dc8401e)

History

#1 Updated by Dominic Cleal about 4 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"}

#2 Updated by Dominic Cleal about 4 years ago

  • Description updated (diff)

#3 Updated by The Foreman Bot over 3 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

#4 Updated by Shimon Shtein over 3 years ago

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

#5 Updated by Ivan Necas over 3 years ago

  • Target version set to 1.7.0

#6 Updated by Dominic Cleal over 3 years ago

  • Legacy Backlogs Release (now unused) set to 175

Also available in: Atom PDF