Bug 1792267

Summary: Custom reports have file extension ".text"
Product: Red Hat Satellite Reporter: Lukáš Hellebrandt <lhellebr>
Component: ReportingAssignee: Marek Hulan <mhulan>
Status: CLOSED CURRENTRELEASE QA Contact: Lukáš Hellebrandt <lhellebr>
Severity: low Docs Contact:
Priority: unspecified    
Version: 6.7.0CC: bkearney, mhulan, oprazak, zhunting
Target Milestone: UnspecifiedKeywords: Triaged
Target Release: Unused   
Hardware: Unspecified   
OS: Unspecified   
Whiteboard:
Fixed In Version: foreman-1.24.1.5-1 Doc Type: If docs needed, set a value
Doc Text:
Story Points: ---
Clone Of: Environment:
Last Closed: 2020-04-14 19:08:41 UTC Type: Bug
Regression: --- Mount Type: ---
Documentation: --- CRM:
Verified Versions: Category: ---
oVirt Team: --- RHEL 7.3 requirements from Atomic Host:
Cloudforms Team: --- Target Upstream Version:
Embargoed:

Description Lukáš Hellebrandt 2020-01-17 12:45:23 UTC 
Description of problem:
Reports that don't use report_render macro have a plaintext format by default when generated. The downloaded file has the extenstion of ".text" which is unusual for text files.

Version-Release number of selected component (if applicable):
Sat 6.7; technically a regression because the extension was .txt in 6.6 but it's not specified what extension the text file should have

How reproducible:
Deterministic

Steps to Reproduce:
1. Create a custom report that doesn't have string 'report_render' in template
2. Generate it

Actual results:
A <report>.text file is downloaded

Expected results:
A <report>.txt file is downloaded
Comment 3 Marek Hulan 2020-01-21 08:51:45 UTC 
Created redmine issue https://projects.theforeman.org/issues/28815 from this bug
Comment 4 Bryan Kearney 2020-01-21 09:06:04 UTC 
Upstream bug assigned to mhulan
Comment 5 Bryan Kearney 2020-01-21 09:06:07 UTC 
Upstream bug assigned to mhulan
Comment 6 Bryan Kearney 2020-01-22 11:06:05 UTC 
Moving this bug to POST for triage into Satellite 6 since the upstream issue https://projects.theforeman.org/issues/28815 has been resolved.
Comment 8 Lukáš Hellebrandt 2020-03-06 15:24:52 UTC 
Verified with Sat 6.7 snap 14 using reproducer from OP.
Comment 9 Bryan Kearney 2020-04-14 19:08:41 UTC 
This was fixed in 6.7.