Fix some image processing (#3313)

* Improve thunmbnail generation

* Respect orientation in Firefox
This commit is contained in:
MeiMei 2018-11-19 09:17:16 +09:00 committed by syuilo
parent a390e57dff
commit 644bc985e7
2 changed files with 29 additions and 6 deletions

View file

@ -65,5 +65,6 @@ export default Vue.extend({
max-height 100% max-height 100%
margin auto margin auto
cursor zoom-out cursor zoom-out
image-orientation from-image
</style> </style>

View file

@ -25,15 +25,33 @@ const log = debug('misskey:drive:add-file');
async function save(path: string, name: string, type: string, hash: string, size: number, metadata: any): Promise<IDriveFile> { async function save(path: string, name: string, type: string, hash: string, size: number, metadata: any): Promise<IDriveFile> {
let thumbnail: Buffer; let thumbnail: Buffer;
let thumbnailExt = 'jpg';
let thumbnailType = 'image/jpeg';
if (['image/jpeg', 'image/png', 'image/webp'].includes(type)) { if (['image/jpeg', 'image/webp'].includes(type)) {
thumbnail = await sharp(path) thumbnail = await sharp(path)
.resize(300) .resize(498, 280, {
fit: 'inside',
withoutEnlargement: true
})
.rotate()
.jpeg({ .jpeg({
quality: 50, quality: 85,
progressive: true progressive: true
}) })
.toBuffer(); .toBuffer();
} else if (['image/png'].includes(type)) {
thumbnail = await sharp(path)
.resize(498, 280, {
fit: 'inside',
withoutEnlargement: true
})
.rotate()
.png()
.toBuffer();
thumbnailExt = 'png';
thumbnailType = 'image/png';
} }
if (config.drive && config.drive.storage == 'minio') { if (config.drive && config.drive.storage == 'minio') {
@ -48,7 +66,7 @@ async function save(path: string, name: string, type: string, hash: string, size
} }
const key = `${config.drive.prefix}/${uuid.v4()}${ext}`; const key = `${config.drive.prefix}/${uuid.v4()}${ext}`;
const thumbnailKey = `${config.drive.prefix}/${uuid.v4()}.jpg`; const thumbnailKey = `${config.drive.prefix}/${uuid.v4()}.${thumbnailExt}`;
const baseUrl = config.drive.baseUrl const baseUrl = config.drive.baseUrl
|| `${ config.drive.config.useSSL ? 'https' : 'http' }://${ config.drive.config.endPoint }${ config.drive.config.port ? `:${config.drive.config.port}` : '' }/${ config.drive.bucket }`; || `${ config.drive.config.useSSL ? 'https' : 'http' }://${ config.drive.config.endPoint }${ config.drive.config.port ? `:${config.drive.config.port}` : '' }/${ config.drive.bucket }`;
@ -60,7 +78,7 @@ async function save(path: string, name: string, type: string, hash: string, size
if (thumbnail) { if (thumbnail) {
await minio.putObject(config.drive.bucket, thumbnailKey, thumbnail, size, { await minio.putObject(config.drive.bucket, thumbnailKey, thumbnail, size, {
'Content-Type': 'image/jpeg', 'Content-Type': thumbnailType,
'Cache-Control': 'max-age=31536000, immutable' 'Cache-Control': 'max-age=31536000, immutable'
}); });
} }
@ -107,7 +125,7 @@ async function save(path: string, name: string, type: string, hash: string, size
await new Promise<IDriveFile>((resolve, reject) => { await new Promise<IDriveFile>((resolve, reject) => {
const writeStream = thumbnailBucket.openUploadStream(name, { const writeStream = thumbnailBucket.openUploadStream(name, {
contentType: 'image/jpeg', contentType: thumbnailType,
metadata: { metadata: {
originalId: file._id originalId: file._id
} }
@ -149,6 +167,10 @@ async function deleteOldFile(user: IRemoteUser) {
* @param comment Comment * @param comment Comment
* @param folderId Folder ID * @param folderId Folder ID
* @param force If set to true, forcibly upload the file even if there is a file with the same hash. * @param force If set to true, forcibly upload the file even if there is a file with the same hash.
* @param isLink Do not save file to local
* @param url URL of source (URLからアップロードされた場合(/)URL)
* @param uri URL of source (URLからアップロードされた場合の元URL)
* @param sensitive Mark file as sensitive
* @return Created drive file * @return Created drive file
*/ */
export default async function( export default async function(