fix: Don't return inputBuf if input is of type PDF

[KararTY]

Pull request checklist:

• CHANGELOG.md was updated, if applicable

Description

Related to #374.

I have a maxThumbnailSize that is very high, which means that libvips default sizing for PDFs is within the limit, which is allowing this if check to return early:

api/pkg/thumbnail/thumbnail.go

Lines 53 to 57 in 67a1371

	// Only resize if the original image has bigger dimensions than maxThumbnailSize
	if image.Width() <= maxThumbnailSize && image.Height() <= maxThumbnailSize {
	// We don't need to resize image nor does it need to be passed through govips.
	return inputBuf, nil
	}

That early return of the inputBuf makes this happen:

Which definitely isn't intentional.

This PR adds an additional conditional to the if check and moves some things around to make the format variable work without crashing if image errors out. (Calling image.Format() when previous image, err := ... fails crashes the software, I learned.)

Images

Before:

After:

And yes, I have thumbnails set to size 900 in Chatterino.

[codecov]

Codecov Report

Merging #377 (26e42f2) into master (a67a14d) will not change coverage.
The diff coverage is 0.00%.

@@           Coverage Diff           @@
##           master     #377   +/-   ##
=======================================
  Coverage   45.34%   45.34%           
=======================================
  Files         100      100           
  Lines        3670     3670           
=======================================
  Hits         1664     1664           
  Misses       1959     1959           
  Partials       47       47           

Impacted Files	Coverage Δ
pkg/thumbnail/thumbnail.go	0.00% <0.00%> (ø)

📣 We’re building smart automated test selection to slash your CI/CD build times. Learn more