Project

General

Profile

Bug #30138

Backslash escapes when downloading a JSON-formatted report multiple times

Added by Ondřej Ezr 4 months ago. Updated about 2 months ago.

Status:
Closed
Priority:
Normal
Assignee:
Category:
Reporting
Target version:
-
Difficulty:
Triaged:
No
Bugzilla link:

Description

Cloned from https://bugzilla.redhat.com/show_bug.cgi?id=1815146

Description of problem:
When I generate a report in JSON format, Submit button gets me to the page where I can download it. It download automatically and the file is correct. When I click the Download button, however, I get the same file but with quotation marks escaped by backslashes, i.e.:

1st time:
..."Organization":"Default Organization","Lifecycle Environment":"Library"...

n+1st time:
...\"Organization\":\"Default Organization\",\"Lifecycle Environment\":\"Library\"...

Version-Release number of selected component (if applicable):
Reproduced with Sat 6.7 snap 16; JSON format not supported in Sat 6.6

How reproducible:
Deterministic

Steps to Reproduce:
1. Have a Satellite with some host
2. Monitor -> Report Templates -> generate Host Statuses
3. Select JSON format, Submit
4. Download the file
5. Click Download and download the file again

Actual results:
Quotation marks escaped

Expected results:
Correct JSON file

Additional info:
This doesn't happen with custom templates containing quotation marks

Associated revisions

Revision 570298b2 (diff)
Added by Ondřej Ezr 4 months ago

Fixes #30138 - report template download button

History

#1 Updated by The Foreman Bot 4 months ago

  • Assignee set to Ondřej Ezr
  • Status changed from New to Ready For Testing
  • Pull request https://github.com/theforeman/foreman/pull/7752 added

#2 Updated by The Foreman Bot 4 months ago

  • Fixed in Releases 2.2.0 added

#3 Updated by The Foreman Bot 4 months ago

  • Pull request https://github.com/theforeman/foreman/pull/7758 added

#4 Updated by Ondřej Ezr 4 months ago

  • Status changed from Ready For Testing to Closed

#5 Updated by Tomer Brisker 4 months ago

  • Category changed from Statistics to Reporting
  • Fixed in Releases 2.1.0 added
  • Fixed in Releases deleted (2.2.0)

#6 Updated by The Foreman Bot about 2 months ago

  • Pull request https://github.com/theforeman/foreman/pull/7975 added

Also available in: Atom PDF