Bug 62545 - Don't use colon in JSON renderer to display indentation
Summary: Don't use colon in JSON renderer to display indentation
Status: RESOLVED FIXED
Alias: None
Product: JMeter
Classification: Unclassified
Component: Main (show other bugs)
Version: 4.0
Hardware: All All
: P2 minor (vote)
Target Milestone: JMETER_5.0
Assignee: JMeter issues mailing list
URL:
Keywords: FixedInTrunk
Depends on:
Blocks:
 
Reported: 2018-07-16 16:44 UTC by Felix Schumacher
Modified: 2018-09-05 14:23 UTC (History)
1 user (show)



Attachments
remove the colon from the "tab" string (1.20 KB, patch)
2018-07-16 16:44 UTC, Felix Schumacher
Details | Diff

Note You need to log in before you can comment on or make changes to this bug.
Description Felix Schumacher 2018-07-16 16:44:33 UTC
Created attachment 36033 [details]
remove the colon from the "tab" string

The RenderAsJSON component uses ":    " as indentation string for a "tab". This makes the rendered JSON string an invalid JSON string.

It is better to use a string of four spaces to build the indentation as the resulting string is then valid JSON.
Comment 1 Felix Schumacher 2018-07-16 16:48:54 UTC
Date: Mon Jul 16 16:48:21 2018
New Revision: 1836054

URL: http://svn.apache.org/viewvc?rev=1836054&view=rev
Log:
Don't use a colon as part of the "tab" string when indenting JSON in RenderAsJSON.

Bugzilla Id: 62545

Modified:
    jmeter/trunk/src/components/org/apache/jmeter/visualizers/RenderAsJSON.java
    jmeter/trunk/test/src/org/apache/jmeter/visualizers/TestRenderAsJson.java
    jmeter/trunk/xdocs/changes.xml