2018-12-13 20:25:24 +07:00
|
|
|
const should = require('should');
|
2021-07-07 19:11:19 +01:00
|
|
|
const sinon = require('sinon');
|
2021-10-16 16:26:05 +01:00
|
|
|
const storage = require('../../../../../core/server/adapters/storage');
|
|
|
|
const activeTheme = require('../../../../../core/frontend/services/theme-engine/active');
|
|
|
|
const handleImageSizes = require('../../../../../core/frontend/web/middleware/handle-image-sizes.js');
|
🎨 Reduced favicon requirements and added image formatting (#14918)
fixes https://github.com/TryGhost/Team/issues/1652
fixes https://github.com/TryGhost/Ghost/issues/13319
**Image formatting**
Added support for changing the format of images via the `handle-image-sizes` middleware (e.g. format SVG to png, jpeg, webp)
This change was required:
- Not all browsers support SVG favicons, so we need to convert them to PNGs
- We can't fit image resizing and formatting in the `serve-favicon` middleware: we need to store the resized image to avoid resizing on every request. This system was already present in the `handle-image-sizes` middleware.
To format an uploaded image:
- Original URL: https://localhost/blog/content/images/2022/05/giphy.gif
- To resize: https://localhost/blog/content/images/size/w256h256/2022/05/giphy.gif (already supported)
- To resize and format to webp: https://localhost/blog/content/images/size/w256h256/format/webp/2022/05/giphy.gif
- Animations are preserved when converting Gifs to Webp and in reverse, and also when only resizing (https://github.com/TryGhost/Ghost/issues/13319)
**Favicons**
- Custom favicons are no longer served via `/favicon.png` or `/favicon.ico` (only for default favicon), but use their full path
- Added support for uploading more image extensions in Ghost as a favicon: .jpg, .jpeg, .gif, .webp and .svg are now supported (already supported .png and .ico).
- File extensions other than jpg/jpeg, png, or ico will always get transformed to the image/png format to guarantee browser support (webp and svg images are not yet supported as favicons by all browsers).
For all image formats, other than .ico files:
- Allowed to upload images larger than 1000px in width and height, they will get cropped to 256x256px.
- Allowed uploading favicons that are not square. They will get cropped automatically.
- Allowed to upload larger files, up to 20MB (will get served at a lower file size after being resized)
For .svg files:
- The minimum size of 60x60px is no longer required.
For .ico files:
- The file size limit is increased to 200kb (coming from 100kb)
2022-05-27 16:36:53 +02:00
|
|
|
const imageTransform = require('@tryghost/image-transform');
|
2018-12-13 20:25:24 +07:00
|
|
|
|
|
|
|
// @TODO make these tests lovely and non specific to implementation
|
|
|
|
describe('handleImageSizes middleware', function () {
|
|
|
|
it('calls next immediately if the url does not match /size/something/', function (done) {
|
|
|
|
const fakeReq = {
|
|
|
|
url: '/size/something'
|
|
|
|
};
|
|
|
|
// CASE: second thing middleware does is try to match to a regex
|
|
|
|
fakeReq.url.match = function () {
|
|
|
|
throw new Error('Should have exited immediately');
|
|
|
|
};
|
|
|
|
handleImageSizes(fakeReq, {}, function next() {
|
|
|
|
done();
|
|
|
|
});
|
|
|
|
});
|
|
|
|
|
|
|
|
it('calls next immediately if the url does not match /size/something/', function (done) {
|
|
|
|
const fakeReq = {
|
|
|
|
url: '/url/whatever/'
|
|
|
|
};
|
|
|
|
// CASE: second thing middleware does is try to match to a regex
|
|
|
|
fakeReq.url.match = function () {
|
|
|
|
throw new Error('Should have exited immediately');
|
|
|
|
};
|
|
|
|
handleImageSizes(fakeReq, {}, function next() {
|
|
|
|
done();
|
|
|
|
});
|
|
|
|
});
|
|
|
|
|
|
|
|
it('calls next immediately if the url does not match /size/something/', function (done) {
|
|
|
|
const fakeReq = {
|
|
|
|
url: '/size//'
|
|
|
|
};
|
|
|
|
// CASE: second thing middleware does is try to match to a regex
|
|
|
|
fakeReq.url.match = function () {
|
|
|
|
throw new Error('Should have exited immediately');
|
|
|
|
};
|
|
|
|
handleImageSizes(fakeReq, {}, function next() {
|
|
|
|
done();
|
|
|
|
});
|
|
|
|
});
|
2021-07-07 19:11:19 +01:00
|
|
|
|
|
|
|
describe('file handling', function () {
|
|
|
|
let dummyStorage;
|
|
|
|
let dummyTheme;
|
🎨 Reduced favicon requirements and added image formatting (#14918)
fixes https://github.com/TryGhost/Team/issues/1652
fixes https://github.com/TryGhost/Ghost/issues/13319
**Image formatting**
Added support for changing the format of images via the `handle-image-sizes` middleware (e.g. format SVG to png, jpeg, webp)
This change was required:
- Not all browsers support SVG favicons, so we need to convert them to PNGs
- We can't fit image resizing and formatting in the `serve-favicon` middleware: we need to store the resized image to avoid resizing on every request. This system was already present in the `handle-image-sizes` middleware.
To format an uploaded image:
- Original URL: https://localhost/blog/content/images/2022/05/giphy.gif
- To resize: https://localhost/blog/content/images/size/w256h256/2022/05/giphy.gif (already supported)
- To resize and format to webp: https://localhost/blog/content/images/size/w256h256/format/webp/2022/05/giphy.gif
- Animations are preserved when converting Gifs to Webp and in reverse, and also when only resizing (https://github.com/TryGhost/Ghost/issues/13319)
**Favicons**
- Custom favicons are no longer served via `/favicon.png` or `/favicon.ico` (only for default favicon), but use their full path
- Added support for uploading more image extensions in Ghost as a favicon: .jpg, .jpeg, .gif, .webp and .svg are now supported (already supported .png and .ico).
- File extensions other than jpg/jpeg, png, or ico will always get transformed to the image/png format to guarantee browser support (webp and svg images are not yet supported as favicons by all browsers).
For all image formats, other than .ico files:
- Allowed to upload images larger than 1000px in width and height, they will get cropped to 256x256px.
- Allowed uploading favicons that are not square. They will get cropped automatically.
- Allowed to upload larger files, up to 20MB (will get served at a lower file size after being resized)
For .svg files:
- The minimum size of 60x60px is no longer required.
For .ico files:
- The file size limit is increased to 200kb (coming from 100kb)
2022-05-27 16:36:53 +02:00
|
|
|
let resizeFromBufferStub;
|
|
|
|
let buffer;
|
2021-07-07 19:11:19 +01:00
|
|
|
|
|
|
|
this.beforeEach(function () {
|
🎨 Reduced favicon requirements and added image formatting (#14918)
fixes https://github.com/TryGhost/Team/issues/1652
fixes https://github.com/TryGhost/Ghost/issues/13319
**Image formatting**
Added support for changing the format of images via the `handle-image-sizes` middleware (e.g. format SVG to png, jpeg, webp)
This change was required:
- Not all browsers support SVG favicons, so we need to convert them to PNGs
- We can't fit image resizing and formatting in the `serve-favicon` middleware: we need to store the resized image to avoid resizing on every request. This system was already present in the `handle-image-sizes` middleware.
To format an uploaded image:
- Original URL: https://localhost/blog/content/images/2022/05/giphy.gif
- To resize: https://localhost/blog/content/images/size/w256h256/2022/05/giphy.gif (already supported)
- To resize and format to webp: https://localhost/blog/content/images/size/w256h256/format/webp/2022/05/giphy.gif
- Animations are preserved when converting Gifs to Webp and in reverse, and also when only resizing (https://github.com/TryGhost/Ghost/issues/13319)
**Favicons**
- Custom favicons are no longer served via `/favicon.png` or `/favicon.ico` (only for default favicon), but use their full path
- Added support for uploading more image extensions in Ghost as a favicon: .jpg, .jpeg, .gif, .webp and .svg are now supported (already supported .png and .ico).
- File extensions other than jpg/jpeg, png, or ico will always get transformed to the image/png format to guarantee browser support (webp and svg images are not yet supported as favicons by all browsers).
For all image formats, other than .ico files:
- Allowed to upload images larger than 1000px in width and height, they will get cropped to 256x256px.
- Allowed uploading favicons that are not square. They will get cropped automatically.
- Allowed to upload larger files, up to 20MB (will get served at a lower file size after being resized)
For .svg files:
- The minimum size of 60x60px is no longer required.
For .ico files:
- The file size limit is increased to 200kb (coming from 100kb)
2022-05-27 16:36:53 +02:00
|
|
|
buffer = Buffer.from([0]);
|
2021-07-07 19:11:19 +01:00
|
|
|
dummyStorage = {
|
|
|
|
async exists() {
|
|
|
|
return true;
|
|
|
|
},
|
|
|
|
|
|
|
|
read() {
|
🎨 Reduced favicon requirements and added image formatting (#14918)
fixes https://github.com/TryGhost/Team/issues/1652
fixes https://github.com/TryGhost/Ghost/issues/13319
**Image formatting**
Added support for changing the format of images via the `handle-image-sizes` middleware (e.g. format SVG to png, jpeg, webp)
This change was required:
- Not all browsers support SVG favicons, so we need to convert them to PNGs
- We can't fit image resizing and formatting in the `serve-favicon` middleware: we need to store the resized image to avoid resizing on every request. This system was already present in the `handle-image-sizes` middleware.
To format an uploaded image:
- Original URL: https://localhost/blog/content/images/2022/05/giphy.gif
- To resize: https://localhost/blog/content/images/size/w256h256/2022/05/giphy.gif (already supported)
- To resize and format to webp: https://localhost/blog/content/images/size/w256h256/format/webp/2022/05/giphy.gif
- Animations are preserved when converting Gifs to Webp and in reverse, and also when only resizing (https://github.com/TryGhost/Ghost/issues/13319)
**Favicons**
- Custom favicons are no longer served via `/favicon.png` or `/favicon.ico` (only for default favicon), but use their full path
- Added support for uploading more image extensions in Ghost as a favicon: .jpg, .jpeg, .gif, .webp and .svg are now supported (already supported .png and .ico).
- File extensions other than jpg/jpeg, png, or ico will always get transformed to the image/png format to guarantee browser support (webp and svg images are not yet supported as favicons by all browsers).
For all image formats, other than .ico files:
- Allowed to upload images larger than 1000px in width and height, they will get cropped to 256x256px.
- Allowed uploading favicons that are not square. They will get cropped automatically.
- Allowed to upload larger files, up to 20MB (will get served at a lower file size after being resized)
For .svg files:
- The minimum size of 60x60px is no longer required.
For .ico files:
- The file size limit is increased to 200kb (coming from 100kb)
2022-05-27 16:36:53 +02:00
|
|
|
return buffer;
|
2021-07-07 19:11:19 +01:00
|
|
|
},
|
|
|
|
|
|
|
|
async saveRaw(buf, url) {
|
|
|
|
return url;
|
|
|
|
}
|
|
|
|
};
|
|
|
|
|
|
|
|
dummyTheme = {
|
|
|
|
config(key) {
|
|
|
|
if (key === 'image_sizes') {
|
|
|
|
return {
|
|
|
|
l: {
|
|
|
|
width: 1000
|
|
|
|
}
|
|
|
|
};
|
|
|
|
}
|
|
|
|
}
|
|
|
|
};
|
|
|
|
|
|
|
|
sinon.stub(storage, 'getStorage').returns(dummyStorage);
|
|
|
|
sinon.stub(activeTheme, 'get').returns(dummyTheme);
|
🎨 Reduced favicon requirements and added image formatting (#14918)
fixes https://github.com/TryGhost/Team/issues/1652
fixes https://github.com/TryGhost/Ghost/issues/13319
**Image formatting**
Added support for changing the format of images via the `handle-image-sizes` middleware (e.g. format SVG to png, jpeg, webp)
This change was required:
- Not all browsers support SVG favicons, so we need to convert them to PNGs
- We can't fit image resizing and formatting in the `serve-favicon` middleware: we need to store the resized image to avoid resizing on every request. This system was already present in the `handle-image-sizes` middleware.
To format an uploaded image:
- Original URL: https://localhost/blog/content/images/2022/05/giphy.gif
- To resize: https://localhost/blog/content/images/size/w256h256/2022/05/giphy.gif (already supported)
- To resize and format to webp: https://localhost/blog/content/images/size/w256h256/format/webp/2022/05/giphy.gif
- Animations are preserved when converting Gifs to Webp and in reverse, and also when only resizing (https://github.com/TryGhost/Ghost/issues/13319)
**Favicons**
- Custom favicons are no longer served via `/favicon.png` or `/favicon.ico` (only for default favicon), but use their full path
- Added support for uploading more image extensions in Ghost as a favicon: .jpg, .jpeg, .gif, .webp and .svg are now supported (already supported .png and .ico).
- File extensions other than jpg/jpeg, png, or ico will always get transformed to the image/png format to guarantee browser support (webp and svg images are not yet supported as favicons by all browsers).
For all image formats, other than .ico files:
- Allowed to upload images larger than 1000px in width and height, they will get cropped to 256x256px.
- Allowed uploading favicons that are not square. They will get cropped automatically.
- Allowed to upload larger files, up to 20MB (will get served at a lower file size after being resized)
For .svg files:
- The minimum size of 60x60px is no longer required.
For .ico files:
- The file size limit is increased to 200kb (coming from 100kb)
2022-05-27 16:36:53 +02:00
|
|
|
resizeFromBufferStub = sinon.stub(imageTransform, 'resizeFromBuffer').resolves(Buffer.from([]));
|
2021-07-07 19:11:19 +01:00
|
|
|
});
|
|
|
|
|
|
|
|
this.afterEach(function () {
|
|
|
|
sinon.restore();
|
|
|
|
});
|
|
|
|
|
🎨 Reduced favicon requirements and added image formatting (#14918)
fixes https://github.com/TryGhost/Team/issues/1652
fixes https://github.com/TryGhost/Ghost/issues/13319
**Image formatting**
Added support for changing the format of images via the `handle-image-sizes` middleware (e.g. format SVG to png, jpeg, webp)
This change was required:
- Not all browsers support SVG favicons, so we need to convert them to PNGs
- We can't fit image resizing and formatting in the `serve-favicon` middleware: we need to store the resized image to avoid resizing on every request. This system was already present in the `handle-image-sizes` middleware.
To format an uploaded image:
- Original URL: https://localhost/blog/content/images/2022/05/giphy.gif
- To resize: https://localhost/blog/content/images/size/w256h256/2022/05/giphy.gif (already supported)
- To resize and format to webp: https://localhost/blog/content/images/size/w256h256/format/webp/2022/05/giphy.gif
- Animations are preserved when converting Gifs to Webp and in reverse, and also when only resizing (https://github.com/TryGhost/Ghost/issues/13319)
**Favicons**
- Custom favicons are no longer served via `/favicon.png` or `/favicon.ico` (only for default favicon), but use their full path
- Added support for uploading more image extensions in Ghost as a favicon: .jpg, .jpeg, .gif, .webp and .svg are now supported (already supported .png and .ico).
- File extensions other than jpg/jpeg, png, or ico will always get transformed to the image/png format to guarantee browser support (webp and svg images are not yet supported as favicons by all browsers).
For all image formats, other than .ico files:
- Allowed to upload images larger than 1000px in width and height, they will get cropped to 256x256px.
- Allowed uploading favicons that are not square. They will get cropped automatically.
- Allowed to upload larger files, up to 20MB (will get served at a lower file size after being resized)
For .svg files:
- The minimum size of 60x60px is no longer required.
For .ico files:
- The file size limit is increased to 200kb (coming from 100kb)
2022-05-27 16:36:53 +02:00
|
|
|
it('redirects for invalid format extension', function (done) {
|
|
|
|
const fakeReq = {
|
|
|
|
url: '/size/w1000/format/test/image.jpg',
|
|
|
|
originalUrl: '/blog/content/images/size/w1000/format/test/image.jpg'
|
|
|
|
};
|
|
|
|
const fakeRes = {
|
|
|
|
redirect(url) {
|
|
|
|
try {
|
|
|
|
url.should.equal('/blog/content/images/image.jpg');
|
|
|
|
} catch (e) {
|
|
|
|
return done(e);
|
|
|
|
}
|
|
|
|
done();
|
|
|
|
}
|
|
|
|
};
|
|
|
|
handleImageSizes(fakeReq, fakeRes, function next(err) {
|
|
|
|
if (err) {
|
|
|
|
return done(err);
|
|
|
|
}
|
|
|
|
done(new Error('Should not have called next'));
|
|
|
|
});
|
|
|
|
});
|
|
|
|
|
2021-07-07 19:11:19 +01:00
|
|
|
it('returns original URL if file is empty', function (done) {
|
|
|
|
dummyStorage.exists = async function (path) {
|
|
|
|
if (path === '/blank_o.png') {
|
|
|
|
return true;
|
|
|
|
}
|
|
|
|
if (path === '/size/w1000/blank.png') {
|
|
|
|
return false;
|
|
|
|
}
|
|
|
|
};
|
🎨 Reduced favicon requirements and added image formatting (#14918)
fixes https://github.com/TryGhost/Team/issues/1652
fixes https://github.com/TryGhost/Ghost/issues/13319
**Image formatting**
Added support for changing the format of images via the `handle-image-sizes` middleware (e.g. format SVG to png, jpeg, webp)
This change was required:
- Not all browsers support SVG favicons, so we need to convert them to PNGs
- We can't fit image resizing and formatting in the `serve-favicon` middleware: we need to store the resized image to avoid resizing on every request. This system was already present in the `handle-image-sizes` middleware.
To format an uploaded image:
- Original URL: https://localhost/blog/content/images/2022/05/giphy.gif
- To resize: https://localhost/blog/content/images/size/w256h256/2022/05/giphy.gif (already supported)
- To resize and format to webp: https://localhost/blog/content/images/size/w256h256/format/webp/2022/05/giphy.gif
- Animations are preserved when converting Gifs to Webp and in reverse, and also when only resizing (https://github.com/TryGhost/Ghost/issues/13319)
**Favicons**
- Custom favicons are no longer served via `/favicon.png` or `/favicon.ico` (only for default favicon), but use their full path
- Added support for uploading more image extensions in Ghost as a favicon: .jpg, .jpeg, .gif, .webp and .svg are now supported (already supported .png and .ico).
- File extensions other than jpg/jpeg, png, or ico will always get transformed to the image/png format to guarantee browser support (webp and svg images are not yet supported as favicons by all browsers).
For all image formats, other than .ico files:
- Allowed to upload images larger than 1000px in width and height, they will get cropped to 256x256px.
- Allowed uploading favicons that are not square. They will get cropped automatically.
- Allowed to upload larger files, up to 20MB (will get served at a lower file size after being resized)
For .svg files:
- The minimum size of 60x60px is no longer required.
For .ico files:
- The file size limit is increased to 200kb (coming from 100kb)
2022-05-27 16:36:53 +02:00
|
|
|
dummyStorage.read = async function (path) {
|
|
|
|
return Buffer.from([]);
|
|
|
|
};
|
|
|
|
|
|
|
|
const fakeReq = {
|
|
|
|
url: '/size/w1000/blank.png',
|
|
|
|
originalUrl: '/blog/content/images/size/w1000/blank.png'
|
|
|
|
};
|
|
|
|
const fakeRes = {
|
|
|
|
redirect(url) {
|
|
|
|
try {
|
|
|
|
url.should.equal('/blog/content/images/blank.png');
|
|
|
|
} catch (e) {
|
|
|
|
return done(e);
|
|
|
|
}
|
|
|
|
done();
|
|
|
|
}
|
|
|
|
};
|
|
|
|
|
|
|
|
handleImageSizes(fakeReq, fakeRes, function next(err) {
|
|
|
|
if (err) {
|
|
|
|
return done(err);
|
|
|
|
}
|
|
|
|
done(new Error('Should not have called next'));
|
|
|
|
});
|
|
|
|
});
|
|
|
|
|
|
|
|
it('continues if file exists', function (done) {
|
|
|
|
dummyStorage.exists = async function (path) {
|
|
|
|
if (path === '/size/w1000/blank.png') {
|
|
|
|
return true;
|
|
|
|
}
|
|
|
|
};
|
|
|
|
|
|
|
|
const fakeReq = {
|
|
|
|
url: '/size/w1000/blank.png',
|
|
|
|
originalUrl: '/size/w1000/blank.png'
|
|
|
|
};
|
|
|
|
const fakeRes = {
|
|
|
|
redirect(url) {
|
|
|
|
done(new Error('Should not have called redirect'));
|
|
|
|
}
|
|
|
|
};
|
|
|
|
|
|
|
|
handleImageSizes(fakeReq, fakeRes, function next(err) {
|
|
|
|
if (err) {
|
|
|
|
return done(err);
|
|
|
|
}
|
|
|
|
done();
|
|
|
|
});
|
|
|
|
});
|
|
|
|
|
|
|
|
it('uses unoptimizedImageExists if it exists', function (done) {
|
|
|
|
dummyStorage.exists = async function (path) {
|
|
|
|
if (path === '/blank_o.png') {
|
|
|
|
return true;
|
|
|
|
}
|
|
|
|
};
|
|
|
|
const spy = sinon.spy(dummyStorage, 'read');
|
2021-07-07 19:11:19 +01:00
|
|
|
|
|
|
|
const fakeReq = {
|
|
|
|
url: '/size/w1000/blank.png',
|
|
|
|
originalUrl: '/size/w1000/blank.png'
|
|
|
|
};
|
|
|
|
const fakeRes = {
|
|
|
|
redirect(url) {
|
🎨 Reduced favicon requirements and added image formatting (#14918)
fixes https://github.com/TryGhost/Team/issues/1652
fixes https://github.com/TryGhost/Ghost/issues/13319
**Image formatting**
Added support for changing the format of images via the `handle-image-sizes` middleware (e.g. format SVG to png, jpeg, webp)
This change was required:
- Not all browsers support SVG favicons, so we need to convert them to PNGs
- We can't fit image resizing and formatting in the `serve-favicon` middleware: we need to store the resized image to avoid resizing on every request. This system was already present in the `handle-image-sizes` middleware.
To format an uploaded image:
- Original URL: https://localhost/blog/content/images/2022/05/giphy.gif
- To resize: https://localhost/blog/content/images/size/w256h256/2022/05/giphy.gif (already supported)
- To resize and format to webp: https://localhost/blog/content/images/size/w256h256/format/webp/2022/05/giphy.gif
- Animations are preserved when converting Gifs to Webp and in reverse, and also when only resizing (https://github.com/TryGhost/Ghost/issues/13319)
**Favicons**
- Custom favicons are no longer served via `/favicon.png` or `/favicon.ico` (only for default favicon), but use their full path
- Added support for uploading more image extensions in Ghost as a favicon: .jpg, .jpeg, .gif, .webp and .svg are now supported (already supported .png and .ico).
- File extensions other than jpg/jpeg, png, or ico will always get transformed to the image/png format to guarantee browser support (webp and svg images are not yet supported as favicons by all browsers).
For all image formats, other than .ico files:
- Allowed to upload images larger than 1000px in width and height, they will get cropped to 256x256px.
- Allowed uploading favicons that are not square. They will get cropped automatically.
- Allowed to upload larger files, up to 20MB (will get served at a lower file size after being resized)
For .svg files:
- The minimum size of 60x60px is no longer required.
For .ico files:
- The file size limit is increased to 200kb (coming from 100kb)
2022-05-27 16:36:53 +02:00
|
|
|
done(new Error('Should not have called redirect'));
|
|
|
|
}
|
|
|
|
};
|
|
|
|
|
|
|
|
handleImageSizes(fakeReq, fakeRes, function next(err) {
|
|
|
|
if (err) {
|
|
|
|
return done(err);
|
|
|
|
}
|
|
|
|
try {
|
|
|
|
spy.calledOnceWithExactly({path: '/blank_o.png'}).should.be.true();
|
|
|
|
} catch (e) {
|
|
|
|
return done(e);
|
|
|
|
}
|
|
|
|
done();
|
|
|
|
});
|
|
|
|
});
|
|
|
|
|
|
|
|
it('uses unoptimizedImageExists if it exists with formatting', function (done) {
|
|
|
|
dummyStorage.exists = async function (path) {
|
|
|
|
if (path === '/blank_o.png') {
|
|
|
|
return true;
|
|
|
|
}
|
|
|
|
};
|
|
|
|
const spy = sinon.spy(dummyStorage, 'read');
|
|
|
|
|
|
|
|
const fakeReq = {
|
|
|
|
url: '/size/w1000/format/webp/blank.png',
|
|
|
|
originalUrl: '/size/w1000/format/webp/blank.png'
|
|
|
|
};
|
|
|
|
const fakeRes = {
|
|
|
|
redirect(url) {
|
|
|
|
done(new Error('Should not have called redirect'));
|
|
|
|
},
|
|
|
|
type: function () {}
|
|
|
|
};
|
|
|
|
const typeStub = sinon.spy(fakeRes, 'type');
|
|
|
|
|
|
|
|
handleImageSizes(fakeReq, fakeRes, function next(err) {
|
|
|
|
if (err) {
|
|
|
|
return done(err);
|
|
|
|
}
|
|
|
|
try {
|
|
|
|
spy.calledOnceWithExactly({path: '/blank_o.png'}).should.be.true();
|
|
|
|
typeStub.calledOnceWithExactly('webp').should.be.true();
|
|
|
|
} catch (e) {
|
|
|
|
return done(e);
|
|
|
|
}
|
|
|
|
done();
|
|
|
|
});
|
|
|
|
});
|
|
|
|
|
|
|
|
it('skips SVG if not formatted', function (done) {
|
|
|
|
dummyStorage.exists = async function (path) {
|
|
|
|
return false;
|
|
|
|
};
|
|
|
|
|
|
|
|
const fakeReq = {
|
|
|
|
url: '/size/w1000/blank.svg',
|
|
|
|
originalUrl: '/blog/content/images/size/w1000/blank.svg'
|
|
|
|
};
|
|
|
|
const fakeRes = {
|
|
|
|
redirect(url) {
|
|
|
|
try {
|
|
|
|
url.should.equal('/blog/content/images/blank.svg');
|
|
|
|
} catch (e) {
|
|
|
|
return done(e);
|
|
|
|
}
|
|
|
|
done();
|
|
|
|
}
|
|
|
|
};
|
|
|
|
|
|
|
|
handleImageSizes(fakeReq, fakeRes, function next(err) {
|
|
|
|
if (err) {
|
|
|
|
return done(err);
|
|
|
|
}
|
|
|
|
done(new Error('Should not have called next'));
|
|
|
|
});
|
|
|
|
});
|
|
|
|
|
|
|
|
it('skips formatting to ico', function (done) {
|
|
|
|
dummyStorage.exists = async function (path) {
|
|
|
|
return false;
|
|
|
|
};
|
|
|
|
|
|
|
|
const fakeReq = {
|
|
|
|
url: '/size/w1000/format/ico/blank.png',
|
|
|
|
originalUrl: '/blog/content/images/size/w1000/format/ico/blank.png'
|
|
|
|
};
|
|
|
|
const fakeRes = {
|
|
|
|
redirect(url) {
|
|
|
|
try {
|
|
|
|
url.should.equal('/blog/content/images/blank.png');
|
|
|
|
} catch (e) {
|
|
|
|
return done(e);
|
|
|
|
}
|
|
|
|
done();
|
|
|
|
}
|
|
|
|
};
|
|
|
|
|
|
|
|
handleImageSizes(fakeReq, fakeRes, function next(err) {
|
|
|
|
if (err) {
|
|
|
|
return done(err);
|
|
|
|
}
|
|
|
|
done(new Error('Should not have called next'));
|
|
|
|
});
|
|
|
|
});
|
|
|
|
|
|
|
|
it('skips formatting from ico', function (done) {
|
|
|
|
dummyStorage.exists = async function (path) {
|
|
|
|
return false;
|
|
|
|
};
|
|
|
|
|
|
|
|
const fakeReq = {
|
|
|
|
url: '/size/w1000/format/png/blank.ico',
|
|
|
|
originalUrl: '/blog/content/images/size/w1000/format/png/blank.ico'
|
|
|
|
};
|
|
|
|
const fakeRes = {
|
|
|
|
redirect(url) {
|
|
|
|
try {
|
|
|
|
url.should.equal('/blog/content/images/blank.ico');
|
|
|
|
} catch (e) {
|
|
|
|
return done(e);
|
|
|
|
}
|
|
|
|
done();
|
|
|
|
}
|
|
|
|
};
|
|
|
|
|
|
|
|
handleImageSizes(fakeReq, fakeRes, function next(err) {
|
|
|
|
if (err) {
|
|
|
|
return done(err);
|
|
|
|
}
|
|
|
|
done(new Error('Should not have called next'));
|
|
|
|
});
|
|
|
|
});
|
|
|
|
|
|
|
|
it('skips formatting to svg', function (done) {
|
|
|
|
dummyStorage.exists = async function (path) {
|
|
|
|
return false;
|
|
|
|
};
|
|
|
|
|
|
|
|
const fakeReq = {
|
|
|
|
url: '/size/w1000/format/svg/blank.png',
|
|
|
|
originalUrl: '/blog/content/images/size/w1000/format/svg/blank.png'
|
|
|
|
};
|
|
|
|
const fakeRes = {
|
|
|
|
redirect(url) {
|
|
|
|
try {
|
|
|
|
url.should.equal('/blog/content/images/blank.png');
|
|
|
|
} catch (e) {
|
|
|
|
return done(e);
|
|
|
|
}
|
2021-07-07 19:11:19 +01:00
|
|
|
done();
|
|
|
|
}
|
|
|
|
};
|
|
|
|
|
|
|
|
handleImageSizes(fakeReq, fakeRes, function next(err) {
|
|
|
|
if (err) {
|
|
|
|
return done(err);
|
|
|
|
}
|
|
|
|
done(new Error('Should not have called next'));
|
|
|
|
});
|
|
|
|
});
|
🎨 Reduced favicon requirements and added image formatting (#14918)
fixes https://github.com/TryGhost/Team/issues/1652
fixes https://github.com/TryGhost/Ghost/issues/13319
**Image formatting**
Added support for changing the format of images via the `handle-image-sizes` middleware (e.g. format SVG to png, jpeg, webp)
This change was required:
- Not all browsers support SVG favicons, so we need to convert them to PNGs
- We can't fit image resizing and formatting in the `serve-favicon` middleware: we need to store the resized image to avoid resizing on every request. This system was already present in the `handle-image-sizes` middleware.
To format an uploaded image:
- Original URL: https://localhost/blog/content/images/2022/05/giphy.gif
- To resize: https://localhost/blog/content/images/size/w256h256/2022/05/giphy.gif (already supported)
- To resize and format to webp: https://localhost/blog/content/images/size/w256h256/format/webp/2022/05/giphy.gif
- Animations are preserved when converting Gifs to Webp and in reverse, and also when only resizing (https://github.com/TryGhost/Ghost/issues/13319)
**Favicons**
- Custom favicons are no longer served via `/favicon.png` or `/favicon.ico` (only for default favicon), but use their full path
- Added support for uploading more image extensions in Ghost as a favicon: .jpg, .jpeg, .gif, .webp and .svg are now supported (already supported .png and .ico).
- File extensions other than jpg/jpeg, png, or ico will always get transformed to the image/png format to guarantee browser support (webp and svg images are not yet supported as favicons by all browsers).
For all image formats, other than .ico files:
- Allowed to upload images larger than 1000px in width and height, they will get cropped to 256x256px.
- Allowed uploading favicons that are not square. They will get cropped automatically.
- Allowed to upload larger files, up to 20MB (will get served at a lower file size after being resized)
For .svg files:
- The minimum size of 60x60px is no longer required.
For .ico files:
- The file size limit is increased to 200kb (coming from 100kb)
2022-05-27 16:36:53 +02:00
|
|
|
|
|
|
|
it('doesn\'t skip SVGs if formatted to PNG', function (done) {
|
|
|
|
dummyStorage.exists = async function (path) {
|
|
|
|
return false;
|
|
|
|
};
|
|
|
|
|
|
|
|
const fakeReq = {
|
|
|
|
url: '/size/w1000/format/png/blank.svg',
|
|
|
|
originalUrl: '/size/w1000/format/png/blank.svg'
|
|
|
|
};
|
|
|
|
const fakeRes = {
|
|
|
|
redirect(url) {
|
|
|
|
done(new Error('Should not have called redirect'));
|
|
|
|
},
|
|
|
|
type: function () {}
|
|
|
|
};
|
|
|
|
const typeStub = sinon.spy(fakeRes, 'type');
|
|
|
|
|
|
|
|
handleImageSizes(fakeReq, fakeRes, function next(err) {
|
|
|
|
if (err) {
|
|
|
|
return done(err);
|
|
|
|
}
|
|
|
|
try {
|
|
|
|
resizeFromBufferStub.calledOnceWithExactly(buffer, {withoutEnlargement: false, width: 1000, format: 'png'}).should.be.true();
|
|
|
|
typeStub.calledOnceWithExactly('png').should.be.true();
|
|
|
|
} catch (e) {
|
|
|
|
return done(e);
|
|
|
|
}
|
|
|
|
done();
|
|
|
|
});
|
|
|
|
});
|
|
|
|
|
|
|
|
it('can format PNG to WEBP', function (done) {
|
|
|
|
dummyStorage.exists = async function (path) {
|
|
|
|
return false;
|
|
|
|
};
|
|
|
|
dummyStorage.read = async function (path) {
|
|
|
|
return buffer;
|
|
|
|
};
|
|
|
|
|
|
|
|
const fakeReq = {
|
|
|
|
url: '/size/w1000/format/webp/blank.png',
|
|
|
|
originalUrl: '/size/w1000/format/webp/blank.png'
|
|
|
|
};
|
|
|
|
const fakeRes = {
|
|
|
|
redirect(url) {
|
|
|
|
done(new Error('Should not have called redirect'));
|
|
|
|
},
|
|
|
|
type: function () {}
|
|
|
|
};
|
|
|
|
const typeStub = sinon.spy(fakeRes, 'type');
|
|
|
|
|
|
|
|
handleImageSizes(fakeReq, fakeRes, function next(err) {
|
|
|
|
if (err) {
|
|
|
|
return done(err);
|
|
|
|
}
|
|
|
|
try {
|
|
|
|
resizeFromBufferStub.calledOnceWithExactly(buffer, {withoutEnlargement: true, width: 1000, format: 'webp'}).should.be.true();
|
|
|
|
typeStub.calledOnceWithExactly('webp').should.be.true();
|
|
|
|
} catch (e) {
|
|
|
|
return done(e);
|
|
|
|
}
|
|
|
|
done();
|
|
|
|
});
|
|
|
|
});
|
|
|
|
|
|
|
|
it('can format GIF to WEBP', function (done) {
|
|
|
|
dummyStorage.exists = async function (path) {
|
|
|
|
return false;
|
|
|
|
};
|
|
|
|
dummyStorage.read = async function (path) {
|
|
|
|
return buffer;
|
|
|
|
};
|
|
|
|
|
|
|
|
const fakeReq = {
|
|
|
|
url: '/size/w1000/format/webp/blank.gif',
|
|
|
|
originalUrl: '/size/w1000/format/webp/blank.gif'
|
|
|
|
};
|
|
|
|
const fakeRes = {
|
|
|
|
redirect(url) {
|
|
|
|
done(new Error('Should not have called redirect'));
|
|
|
|
},
|
|
|
|
type: function () {}
|
|
|
|
};
|
|
|
|
const typeStub = sinon.spy(fakeRes, 'type');
|
|
|
|
|
|
|
|
handleImageSizes(fakeReq, fakeRes, function next(err) {
|
|
|
|
if (err) {
|
|
|
|
return done(err);
|
|
|
|
}
|
|
|
|
try {
|
|
|
|
resizeFromBufferStub.calledOnceWithExactly(buffer, {withoutEnlargement: true, width: 1000, format: 'webp'}).should.be.true();
|
|
|
|
typeStub.calledOnceWithExactly('webp').should.be.true();
|
|
|
|
} catch (e) {
|
|
|
|
return done(e);
|
|
|
|
}
|
|
|
|
done();
|
|
|
|
});
|
|
|
|
});
|
|
|
|
|
|
|
|
it('can format WEBP to GIF', function (done) {
|
|
|
|
dummyStorage.exists = async function (path) {
|
|
|
|
return false;
|
|
|
|
};
|
|
|
|
dummyStorage.read = async function (path) {
|
|
|
|
return buffer;
|
|
|
|
};
|
|
|
|
|
|
|
|
const fakeReq = {
|
|
|
|
url: '/size/w1000/format/gif/blank.webp',
|
|
|
|
originalUrl: '/size/w1000/format/gif/blank.webp'
|
|
|
|
};
|
|
|
|
const fakeRes = {
|
|
|
|
redirect(url) {
|
|
|
|
done(new Error('Should not have called redirect'));
|
|
|
|
},
|
|
|
|
type: function () {}
|
|
|
|
};
|
|
|
|
const typeStub = sinon.spy(fakeRes, 'type');
|
|
|
|
|
|
|
|
handleImageSizes(fakeReq, fakeRes, function next(err) {
|
|
|
|
if (err) {
|
|
|
|
return done(err);
|
|
|
|
}
|
|
|
|
try {
|
|
|
|
resizeFromBufferStub.calledOnceWithExactly(buffer, {withoutEnlargement: true, width: 1000, format: 'gif'}).should.be.true();
|
|
|
|
typeStub.calledOnceWithExactly('gif').should.be.true();
|
|
|
|
} catch (e) {
|
|
|
|
return done(e);
|
|
|
|
}
|
|
|
|
done();
|
|
|
|
});
|
|
|
|
});
|
2021-07-07 19:11:19 +01:00
|
|
|
});
|
2018-12-13 20:25:24 +07:00
|
|
|
});
|