Skip to content

Commit 58ae7fb

Browse files
committed
fix: S3. Change name generation from Date.now to UUID
closes Visual-Regression-Tracker/Visual-Regression-Tracker#462
1 parent bab7d58 commit 58ae7fb

File tree

3 files changed

+10
-2
lines changed

3 files changed

+10
-2
lines changed

src/static/aws/s3.service.ts

Lines changed: 2 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -4,6 +4,7 @@ import { Static } from '../static.interface';
44
import { DeleteObjectCommand, GetObjectCommand, PutObjectCommand, S3Client } from '@aws-sdk/client-s3';
55
import { Readable } from 'stream';
66
import { getSignedUrl } from '@aws-sdk/s3-request-presigner';
7+
import { generateNewImageName } from '../utils';
78

89
export class AWSS3Service implements Static {
910
private readonly logger: Logger = new Logger(AWSS3Service.name);
@@ -26,7 +27,7 @@ export class AWSS3Service implements Static {
2627
}
2728

2829
async saveImage(type: 'screenshot' | 'diff' | 'baseline', imageBuffer: Buffer): Promise<string> {
29-
const imageName = `${Date.now()}.${type}.png`;
30+
const imageName = generateNewImageName(type);
3031
try {
3132
await this.s3Client.send(
3233
new PutObjectCommand({

src/static/hdd/hdd.service.ts

Lines changed: 2 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -5,12 +5,13 @@ import { PNG, PNGWithMetadata } from 'pngjs';
55
import uuidAPIKey from 'uuid-apikey';
66
import { Static } from '../static.interface';
77
import { HDD_IMAGE_PATH } from './constants';
8+
import { generateNewImageName } from '../utils';
89

910
export class HddService implements Static {
1011
private readonly logger: Logger = new Logger(HddService.name);
1112

1213
generateNewImage(type: 'screenshot' | 'diff' | 'baseline'): { imageName: string; imagePath: string } {
13-
const imageName = `${uuidAPIKey.create({ noDashes: true }).apiKey}.${type}.png`;
14+
const imageName = generateNewImageName(type);
1415
return {
1516
imageName,
1617
imagePath: this.getImagePath(imageName),

src/static/utils.ts

Lines changed: 6 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -1,7 +1,13 @@
1+
import uuidAPIKey from 'uuid-apikey';
2+
13
export function isHddStaticServiceConfigured() {
24
return !process.env.STATIC_SERVICE || process.env.STATIC_SERVICE === 'hdd';
35
}
46

57
export function isS3ServiceConfigured() {
68
return !process.env.STATIC_SERVICE || process.env.STATIC_SERVICE === 's3';
79
}
10+
11+
export function generateNewImageName(type: 'screenshot' | 'diff' | 'baseline'): string {
12+
return`${uuidAPIKey.create({ noDashes: true }).apiKey}.${type}.png`;
13+
}

0 commit comments

Comments
 (0)