Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Support serving images under s3 subdirectories, Fix to make /fit-in/ work; Fix for VipsJpeg: Invalid SOS error plus several other critical fixes. #130

Closed
wants to merge 13 commits into from
Closed

Support serving images under s3 subdirectories, Fix to make /fit-in/ work; Fix for VipsJpeg: Invalid SOS error plus several other critical fixes. #130

Changes from 1 commit
Commits
Show all changes
13 commits
Select commit Hold shift + click to select a range
98c94a4
Support serving images under s3 subdirectories
Jul 24, 2019
fe1bf84
Handle all thumbor URLs when trying to get s3 key
Jul 25, 2019
3fc4fd0
Regex adjustments to pass the Tests
Jul 25, 2019
03f3a1a
Make /fit-in/ directive work again
Jul 25, 2019
7c117fa
Fix VipsJpeg: Invalid SOS parameters for sequential JPEG errors, and …
Jul 27, 2019
ec97cac
Fix issue wherein image is not served if no dimension is set, sorry, …
Jul 27, 2019
7dbf592
Fix for proper resizing if the filename or path contains letter x wit…
Jul 27, 2019
2f1bf56
Fix for proper resizing if the filename or path contains letter x wit…
Jul 27, 2019
4993496
Improve getting the s3 key from various filename conventions
Jul 28, 2019
aae497d
Typos and removing unneeded comments as noted by ahmetcetin39
Jul 29, 2019
63a4f64
Update Runtime to Node12.x, Update Sharp to 0.23.1
Dec 18, 2019
bf10401
Handle incomplete dimensions
Dec 18, 2019
36381ed
Reverting sharp to 0.22.1 due to fit-in/inside behaviour in 0.23.x
Dec 18, 2019
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
Next Next commit
Support serving images under s3 subdirectories
  • Loading branch information
Arpee Ong committed Jul 24, 2019
commit 98c94a4fe9c186d465818881b112c6d508c26df2
13 changes: 8 additions & 5 deletions source/image-handler/image-request.js
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -14,7 +14,7 @@
const ThumborMapping = require('./thumbor-mapping');

class ImageRequest {

/**
* Initializer function for creating a new image request, used by the image
* handler to perform image modifications.
Expand Down Expand Up @@ -137,8 +137,11 @@ class ImageRequest {
return decoded.key;
} else if (requestType === "Thumbor" || requestType === "Custom") {
// Parse the key from the end of the path
const key = (event["path"]).split("/");
return key[key.length - 1];
//const key = (event["path"]).split("/");
Copy link

Choose a reason for hiding this comment

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

Delete these two lines? or add comment?

//return key[key.length - 1];

//Arpee: Support serving images under s3 subdirectories
return decodeURIComponent(event["path"].replace(/\/\d+x\d+/,'').substring(1));
} else {
// Return an error for all other conditions
throw ({
Expand All @@ -160,8 +163,8 @@ class ImageRequest {
const path = event["path"];
// ----
const matchDefault = new RegExp(/^(\/?)([0-9a-zA-Z+\/]{4})*(([0-9a-zA-Z+\/]{2}==)|([0-9a-zA-Z+\/]{3}=))?$/);
const matchThumbor = new RegExp(/^(\/?)((fit-in)?|(filters:.+\(.?\))?|(unsafe)?).*(.+jpg|.+png|.+webp|.+tiff|.+jpeg)$/);
const matchCustom = new RegExp(/(\/?)(.*)(jpg|png|webp|tiff|jpeg)/);
const matchThumbor = new RegExp(/^(\/?)((fit-in)?|(filters:.+\(.?\))?|(unsafe)?).*(.+jpg|.+png|.+webp|.+tiff|.+jpeg)$/i);
const matchCustom = new RegExp(/(\/?)(.*)(jpg|png|webp|tiff|jpeg)/i);
const definedEnvironmentVariables = (
(process.env.REWRITE_MATCH_PATTERN !== "") &&
(process.env.REWRITE_SUBSTITUTION !== "") &&
Expand Down