Skip to content

Conversation

@rullzer
Copy link
Member

@rullzer rullzer commented May 2, 2020

Else the wrong mimetype might be set. Resulting in continious
regeneration of previews when browsing.

Else the wrong mimetype might be set. Resulting in continious
regeneration of previews when browsing.

Signed-off-by: Roeland Jago Douma <roeland@famdouma.nl>
@rullzer
Copy link
Member Author

rullzer commented May 2, 2020

Basically without this all previews default to png when resizing. While we need to respect the original (could be jpg).

And due to this the caching goes boom as well.

@rullzer rullzer mentioned this pull request May 4, 2020
2 tasks
Copy link
Member

@ChristophWurst ChristophWurst left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Makes sense

@ChristophWurst ChristophWurst added 4. to release Ready to be released and/or waiting for tests to finish and removed 3. to review Waiting for reviews labels May 4, 2020
@MorrisJobke MorrisJobke merged commit e102264 into master May 4, 2020
@MorrisJobke MorrisJobke deleted the fix/resize_wrong_mime branch May 4, 2020 08:06
@MorrisJobke
Copy link
Member

@rullzer Backport?

@rullzer
Copy link
Member Author

rullzer commented May 4, 2020

@rullzer Backport?

this bug was introduced by an improvement from @icewind1991 which It hink was not backported.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

4. to release Ready to be released and/or waiting for tests to finish bug feature: previews and thumbnails

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants