Uploaded image for project: 'OpenAM'
  1. OpenAM
  2. OPENAM-12092

Prettyprint for Amster should be a switch

    Details

    • Type: Bug
    • Status: Resolved
    • Priority: Major
    • Resolution: Fixed
    • Affects Version/s: 14.5.0
    • Fix Version/s: 6.0.0
    • Component/s: Amster
    • Environment:
      Amster-5.5.0
    • Target Version/s:
    • Needs backport:
      No
    • Needs QA verification:
      No
    • Functional tests:
      No
    • Are the reproduction steps defined?:
      Yes and I used the same an in the description

      Description

      Bug description

      OPENAM-10641 introduced pretty print for amster output, We should add a switch so that pretty print is optional

      PrettyPrint being used as default  had the unfortunate side-affect of breaking the steps we publish for how to update property values (see https://backstage.forgerock.com/knowledge/kb/article/a73487721

      Previously customers would:

      1. Do a read
      2. copy the output and change the property value needed
      3. remove the _rev and _id
      4. do an update using that amended output as the body
      Expected behaviour

      Step 4 passes and the property is updated

      Current behaviour

      The update step fails because of the line endings introduced by prettyprint. 

      Work around

      Remove the line breaks for example, using sed or an online tool  (this has been added to the KB)

      It would be great if we could provide better instructions than "use sed" perhaps there is a way to override the DefaultPrettyPrinter used by CrestCommandSupport.groovy? 

       

       

        Attachments

          Activity

            People

            • Assignee:
              sean.oneill Sean ONeill [X] (Inactive)
              Reporter:
              philip.anderson Philip Anderson
            • Votes:
              0 Vote for this issue
              Watchers:
              4 Start watching this issue

              Dates

              • Created:
                Updated:
                Resolved: