Project

General

Profile

Bug #15984

Backslashes added to overrriden params when changing environments within host edit

Added by Alejandro Falcon over 2 years ago. Updated 5 months ago.

Status:
Closed
Priority:
Normal
Assignee:
Category:
Smart Variables
Target version:
Difficulty:
Triaged:
Bugzilla link:
Team Backlog:
Fixed in Releases:
Found in Releases:

Description

How to reproduce:

  • Edit a host.
  • Go to params and override any(tried hashes and arrays)
  • Change host environment
  • Overriden param will show with backslashes and \r\n for line breaks.

Related issues

Related to Foreman - Bug #22088: edit hostgroup form reloads parameters on environment changeNew2017-12-27
Related to Foreman - Refactor #24951: No local variable `host` exists.Closed

Associated revisions

Revision 265e1d98 (diff)
Added by Ori Rabin 9 months ago

Fixes #15984 - Avoid escaping overridden parameters (#5126)

History

#1 Updated by Dominic Cleal over 2 years ago

  • Subject changed from Foreman add backslashes to overrriden params when changing environments withing edit screen. to Backslashes added to overrriden params when changing environments within host edit
  • Description updated (diff)
  • Category set to Smart Variables

I'd recommend re-testing with 1.12.1 if you can, there were a couple of fixes in that release (and possibly 1.12.0 itself) that may be applicable.

#2 Updated by Alejandro Falcon over 2 years ago

I've tested with 1.12.2 and still getting the same problem.

#3 Updated by Ori Rabin 12 months ago

  • Bugzilla link set to 1506814

#4 Updated by Ori Rabin 12 months ago

  • Target version set to 233
  • Assignee set to Ori Rabin
  • Status changed from New to Assigned

#5 Updated by The Foreman Bot 12 months ago

  • Status changed from Assigned to Ready For Testing
  • Pull request https://github.com/theforeman/foreman/pull/5126 added

#6 Updated by Ori Rabin 12 months ago

  • Related to Bug #22088: edit hostgroup form reloads parameters on environment change added

#7 Updated by Marek Hulán 9 months ago

  • Legacy Backlogs Release (now unused) set to 330

#8 Updated by Ori Rabin 9 months ago

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

#9 Updated by Lukas Zapletal 3 months ago

Also available in: Atom PDF