X-Git-Url: https://git.immae.eu/?a=blobdiff_plain;ds=sidebyside;f=server%2Flib%2Fredis.ts;h=8aec4b7937baed041d67da68d14b05a84b7284a5;hb=a5ee023cf36cfa11e6088c1f8111e237076474eb;hp=5313c46857a95254df77cc2d4717e763a803db3d;hpb=db48de8597897e5024f8e9ed5acb1a8f40748169;p=github%2FChocobozzz%2FPeerTube.git diff --git a/server/lib/redis.ts b/server/lib/redis.ts index 5313c4685..8aec4b793 100644 --- a/server/lib/redis.ts +++ b/server/lib/redis.ts @@ -1,4 +1,4 @@ -import * as express from 'express' +import express from 'express' import { createClient, RedisClient } from 'redis' import { logger } from '../helpers/logger' import { generateRandomString } from '../helpers/utils' @@ -7,11 +7,13 @@ import { USER_EMAIL_VERIFY_LIFETIME, USER_PASSWORD_RESET_LIFETIME, USER_PASSWORD_CREATE_LIFETIME, - VIDEO_VIEW_LIFETIME, + VIEW_LIFETIME, WEBSERVER, - TRACKER_RATE_LIMITS + TRACKER_RATE_LIMITS, + RESUMABLE_UPLOAD_SESSION_LIFETIME } from '../initializers/constants' import { CONFIG } from '../initializers/config' +import { exists } from '@server/helpers/custom-validators/misc' type CachedRoute = { body: string @@ -84,6 +86,10 @@ class Redis { return generatedString } + async removePasswordVerificationString (userId: number) { + return this.removeValue(this.generateResetPasswordKey(userId)) + } + async getResetPasswordLink (userId: number) { return this.getValue(this.generateResetPasswordKey(userId)) } @@ -115,11 +121,19 @@ class Redis { /* ************ Views per IP ************ */ setIPVideoView (ip: string, videoUUID: string) { - return this.setValue(this.generateViewKey(ip, videoUUID), '1', VIDEO_VIEW_LIFETIME) + return this.setValue(this.generateIPViewKey(ip, videoUUID), '1', VIEW_LIFETIME.VIEW) + } + + setIPVideoViewer (ip: string, videoUUID: string) { + return this.setValue(this.generateIPViewerKey(ip, videoUUID), '1', VIEW_LIFETIME.VIEWER) } async doesVideoIPViewExist (ip: string, videoUUID: string) { - return this.exists(this.generateViewKey(ip, videoUUID)) + return this.exists(this.generateIPViewKey(ip, videoUUID)) + } + + async doesVideoIPViewerExist (ip: string, videoUUID: string) { + return this.exists(this.generateIPViewerKey(ip, videoUUID)) } /* ************ Tracker IP block ************ */ @@ -151,65 +165,132 @@ class Redis { return this.setObject(this.generateCachedRouteKey(req), cached, lifetime) } - /* ************ Video views ************ */ + /* ************ Video views stats ************ */ - addVideoView (videoId: number) { - const keyIncr = this.generateVideoViewKey(videoId) - const keySet = this.generateVideosViewKey() + addVideoViewStats (videoId: number) { + const { videoKey, setKey } = this.generateVideoViewStatsKeys({ videoId }) return Promise.all([ - this.addToSet(keySet, videoId.toString()), - this.increment(keyIncr) + this.addToSet(setKey, videoId.toString()), + this.increment(videoKey) ]) } - async getVideoViews (videoId: number, hour: number) { - const key = this.generateVideoViewKey(videoId, hour) + async getVideoViewsStats (videoId: number, hour: number) { + const { videoKey } = this.generateVideoViewStatsKeys({ videoId, hour }) - const valueString = await this.getValue(key) + const valueString = await this.getValue(videoKey) const valueInt = parseInt(valueString, 10) if (isNaN(valueInt)) { - logger.error('Cannot get videos views of video %d in hour %d: views number is NaN (%s).', videoId, hour, valueString) + logger.error('Cannot get videos views stats of video %d in hour %d: views number is NaN (%s).', videoId, hour, valueString) return undefined } return valueInt } - async getVideosIdViewed (hour: number) { - const key = this.generateVideosViewKey(hour) + async listVideosViewedForStats (hour: number) { + const { setKey } = this.generateVideoViewStatsKeys({ hour }) - const stringIds = await this.getSet(key) + const stringIds = await this.getSet(setKey) return stringIds.map(s => parseInt(s, 10)) } - deleteVideoViews (videoId: number, hour: number) { - const keySet = this.generateVideosViewKey(hour) - const keyIncr = this.generateVideoViewKey(videoId, hour) + deleteVideoViewsStats (videoId: number, hour: number) { + const { setKey, videoKey } = this.generateVideoViewStatsKeys({ videoId, hour }) + + return Promise.all([ + this.deleteFromSet(setKey, videoId.toString()), + this.deleteKey(videoKey) + ]) + } + + /* ************ Local video views buffer ************ */ + + addLocalVideoView (videoId: number) { + const { videoKey, setKey } = this.generateLocalVideoViewsKeys(videoId) return Promise.all([ - this.deleteFromSet(keySet, videoId.toString()), - this.deleteKey(keyIncr) + this.addToSet(setKey, videoId.toString()), + this.increment(videoKey) ]) } + async getLocalVideoViews (videoId: number) { + const { videoKey } = this.generateLocalVideoViewsKeys(videoId) + + const valueString = await this.getValue(videoKey) + const valueInt = parseInt(valueString, 10) + + if (isNaN(valueInt)) { + logger.error('Cannot get videos views of video %d: views number is NaN (%s).', videoId, valueString) + return undefined + } + + return valueInt + } + + async listLocalVideosViewed () { + const { setKey } = this.generateLocalVideoViewsKeys() + + const stringIds = await this.getSet(setKey) + return stringIds.map(s => parseInt(s, 10)) + } + + deleteLocalVideoViews (videoId: number) { + const { setKey, videoKey } = this.generateLocalVideoViewsKeys(videoId) + + return Promise.all([ + this.deleteFromSet(setKey, videoId.toString()), + this.deleteKey(videoKey) + ]) + } + + /* ************ Resumable uploads final responses ************ */ + + setUploadSession (uploadId: string, response?: { video: { id: number, shortUUID: string, uuid: string } }) { + return this.setValue( + 'resumable-upload-' + uploadId, + response + ? JSON.stringify(response) + : '', + RESUMABLE_UPLOAD_SESSION_LIFETIME + ) + } + + doesUploadSessionExist (uploadId: string) { + return this.exists('resumable-upload-' + uploadId) + } + + async getUploadSession (uploadId: string) { + const value = await this.getValue('resumable-upload-' + uploadId) + + return value + ? JSON.parse(value) + : '' + } + + deleteUploadSession (uploadId: string) { + return this.deleteKey('resumable-upload-' + uploadId) + } + /* ************ Keys generation ************ */ generateCachedRouteKey (req: express.Request) { return req.method + '-' + req.originalUrl } - private generateVideosViewKey (hour?: number) { - if (!hour) hour = new Date().getHours() - - return `videos-view-h${hour}` + private generateLocalVideoViewsKeys (videoId?: Number) { + return { setKey: `local-video-views-buffer`, videoKey: `local-video-views-buffer-${videoId}` } } - private generateVideoViewKey (videoId: number, hour?: number) { - if (!hour) hour = new Date().getHours() + private generateVideoViewStatsKeys (options: { videoId?: number, hour?: number }) { + const hour = exists(options.hour) + ? options.hour + : new Date().getHours() - return `video-view-${videoId}-h${hour}` + return { setKey: `videos-view-h${hour}`, videoKey: `video-view-${options.videoId}-h${hour}` } } private generateResetPasswordKey (userId: number) { @@ -220,10 +301,14 @@ class Redis { return 'verify-email-' + userId } - private generateViewKey (ip: string, videoUUID: string) { + private generateIPViewKey (ip: string, videoUUID: string) { return `views-${videoUUID}-${ip}` } + private generateIPViewerKey (ip: string, videoUUID: string) { + return `viewer-${videoUUID}-${ip}` + } + private generateTrackerBlockIPKey (ip: string) { return `tracker-block-ip-${ip}` } @@ -255,7 +340,7 @@ class Redis { } private addToSet (key: string, value: string) { - return new Promise((res, rej) => { + return new Promise((res, rej) => { this.client.sadd(this.prefix + key, value, err => err ? rej(err) : res()) }) } @@ -290,6 +375,16 @@ class Redis { }) } + private removeValue (key: string) { + return new Promise((res, rej) => { + this.client.del(this.prefix + key, err => { + if (err) return rej(err) + + return res() + }) + }) + } + private setObject (key: string, obj: { [id: string]: string }, expirationMilliseconds: number) { return new Promise((res, rej) => { this.client.hmset(this.prefix + key, obj, (err, ok) => {