Project

General

Profile

Bug #36028

assets are not compressed during delivery anymore

Added by Evgeni Golov about 2 months ago. Updated about 1 month ago.

Status:
Closed
Priority:
Normal
Assignee:
Category:
Foreman modules
Target version:

Description

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

Description of problem:

# curl -k -I -H "Accept-Encoding: gzip" https://localhost/webpack/vendor-950b729ed5965c2a80e1.js
HTTP/2 200 
date: Wed, 01 Feb 2023 09:10:12 GMT
server: Apache
last-modified: Tue, 24 Jan 2023 16:16:09 GMT
etag: "a481-5f304d562e440" 
accept-ranges: bytes
content-length: 42113
content-type: application/javascript

Version-Release number of selected component (if applicable):
6.13.0

How reproducible:
100%

Steps to Reproduce:
1. curl -k -I -H "Accept-Encoding: gzip" <URL_of_asset>

Actual results:
- No "content-encoding: gzip" header
- content-length shows the "full" size of the file, not the gziped size

Expected results:

# curl -k -I -H "Accept-Encoding: gzip" https://localhost/webpack/vendor-950b729ed5965c2a80e1.js
HTTP/2 200 
date: Wed, 01 Feb 2023 09:12:25 GMT
server: Apache
vary: Accept-Encoding
last-modified: Tue, 24 Jan 2023 16:16:09 GMT
etag: "3748-5f304d562e440" 
accept-ranges: bytes
content-length: 14152
content-encoding: gzip
content-type: text/javascript

Additional info:

Associated revisions

Revision dee223bb (diff)
Added by Evgeni Golov about 2 months ago

Fixes #36028 - ensure compressed assets are returned if available

Fixes: 6a02be6e8faecb4210f16a8a5aea37397bfe3f74

History

#1 Updated by The Foreman Bot about 2 months ago

  • Assignee set to Evgeni Golov
  • Status changed from New to Ready For Testing
  • Pull request https://github.com/theforeman/puppet-foreman/pull/1106 added

#2 Updated by Evgeni Golov about 2 months ago

  • Description updated (diff)
  • Subject changed from assets are not compressed during delivery anymore to assets are not compressed during delivery anymore

#3 Updated by Evgeni Golov about 2 months ago

  • Target version set to 3.5.2
  • Found in Releases 3.5.0, 3.5.1, 3.6.0 added

#4 Updated by The Foreman Bot about 2 months ago

  • Fixed in Releases 3.6.0 added

#5 Updated by Evgeni Golov about 2 months ago

  • Status changed from Ready For Testing to Closed

#6 Updated by The Foreman Bot about 2 months ago

  • Pull request https://github.com/theforeman/foreman-installer/pull/830 added

#7 Updated by Evgeni Golov about 2 months ago

  • Fixed in Releases 3.5.2 added

#8 Updated by Ewoud Kohl van Wijngaarden about 1 month ago

  • Triaged changed from No to Yes
  • Category set to Foreman modules

Also available in: Atom PDF