Skip to content
This repository has been archived by the owner on Nov 27, 2023. It is now read-only.

Problem when used with Minify HTML extension #18

Closed
saltpeanuts opened this issue Jan 13, 2017 · 5 comments
Closed

Problem when used with Minify HTML extension #18

saltpeanuts opened this issue Jan 13, 2017 · 5 comments

Comments

@saltpeanuts
Copy link

saltpeanuts commented Jan 13, 2017

When used alongside the Minify HTML extension: https://github.com/nlemoine/bolt-extension-minify-html

Srcset images are not loaded. Looking at the network monitor I'm seeing a 500 error for each image.

Images are still being generated and cached.

The original passthrough image is loading fine.

Did a very quick comparison of the minified vs original srcset markup to see if it's getting mangled somewhere by the minifier, but nothing immediately jumps out as being obviously out of place. Only had time for a very brief look, though.

@cdowdy
Copy link
Owner

cdowdy commented Jan 13, 2017

@saltpeanuts this isn't an error for this extension.

if you open the image in a separate tab you'll get:

LogicException in StreamedResponse.php line 124: The content cannot be set on a StreamedResponse instance.

and I modified it to actually run in debug mode and you'll get:

minify-error

This uses Symfony StreamedResponse and sets a notModified header so you'll need to open a bug over at the minify repo

can show them these if you'd like:

symfony/symfony#15564

symfony/symfony#17036

@cdowdy cdowdy closed this as completed Jan 13, 2017
@saltpeanuts
Copy link
Author

@cdowdy oh, totally. I knew it wasn't a boltbetterthumbs bug exactly, just an issue when they're both used together so thought you'd still want a heads up. They're both common tasks so not unusual for people to run them at the same time. I'll let @nlemoine know too.

@cdowdy
Copy link
Owner

cdowdy commented Jan 13, 2017

oh alright.. haha I saw the bug report and thought I messed up something... Thanks for the heads up!

@cdowdy
Copy link
Owner

cdowdy commented Jan 14, 2017

hey @saltpeanuts I decided to tinker a bit and have something that might work you wanna take a look?

@cdowdy cdowdy reopened this Jan 14, 2017
@cdowdy
Copy link
Owner

cdowdy commented Jan 14, 2017

@saltpeanuts err.... nevermind I actually just went to the minify repo and saw they put a check in for a streamed response... nevermind haha.. Thanks though!

@cdowdy cdowdy closed this as completed Jan 14, 2017
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

No branches or pull requests

2 participants