Suwayomi / Suwayomi-Server

A rewrite of Tachiyomi for the Desktop
Mozilla Public License 2.0
3.93k stars 201 forks source link

[Bug] Invalid images return with the content-type image/.tmp #129

Closed Syer10 closed 2 years ago

Syer10 commented 3 years ago

Device information

Steps to reproduce

  1. Open Dynasty-Issues Source from Dynasty Extension
  2. Click broken manga such as Nico Nico Yuri Hime

Expected behavior

Request should return 404 or similar error with no content-type header

Actual behavior

Request returns invalid content with a header image/.tmp

Other details

The same header is also used for covers that were made with 0.4.1 and below, but they return valid images in the content

Syer10 commented 3 years ago

Just tested it a bit more, it also seems to happen on every second load of a image

AriaMoradi commented 3 years ago

This issue was fixed (#100), your cached images are saved with .tmp extension which in turn CachedImageRequest returns the same. delete your image cache and it'll be fixed, although it returning patterns like image/.jpeg and image/.png is wrong and also fixed that in a29010e0d7030b03976f888a8b361db78a79a42a

AriaMoradi commented 3 years ago

request for thumbnail of Nico Nico Yuri Hime Oct. 2014 returns a text/html content response which breaks Tachidesk on windows and linux. On windows, renaming the .tmp file will fail and on linux files like 17.html; charset=utf-8 are created

AriaMoradi commented 3 years ago

does this issue still persist?

Syer10 commented 3 years ago

Yes

AriaMoradi commented 2 years ago

this was fixed in #100, opening another issue for https://github.com/Suwayomi/Tachidesk-Server/issues/129#issuecomment-854504427

Farzout commented 1 month ago

it did not fix my problem with deleting cashe or temp folder on windows

Edit: my problem was fixed clearing my browser cache thx

Robonau commented 1 month ago

it did not fix my problem with deleting cashe or temp folder on windows

please make a new issue rather than commenting on an issue fixed and closed 3 years ago