X-Git-Url: https://git.immae.eu/?a=blobdiff_plain;f=server%2Fhelpers%2Fcustom-validators%2Faccounts.ts;h=191de1496eec4b545e04a0438a1b85f95881b986;hb=8a19bee1a1ee39f973bb37429e4f73c3f2873cdb;hp=cc8641d6b40912f316823ae8014fee6c69daa9af;hpb=e8cb44090e654fda339506dccfcec7fea8722723;p=github%2FChocobozzz%2FPeerTube.git diff --git a/server/helpers/custom-validators/accounts.ts b/server/helpers/custom-validators/accounts.ts index cc8641d6b..191de1496 100644 --- a/server/helpers/custom-validators/accounts.ts +++ b/server/helpers/custom-validators/accounts.ts @@ -4,16 +4,22 @@ import 'express-validator' import * as validator from 'validator' import { AccountModel } from '../../models/account/account' import { isUserDescriptionValid, isUserUsernameValid } from './users' +import { exists } from './misc' +import { CONFIG } from '../../initializers' function isAccountNameValid (value: string) { return isUserUsernameValid(value) } +function isAccountIdValid (value: string) { + return exists(value) +} + function isAccountDescriptionValid (value: string) { return isUserDescriptionValid(value) } -function isAccountIdExist (id: number | string, res: Response) { +function isAccountIdExist (id: number | string, res: Response, sendNotFound = true) { let promise: Bluebird if (validator.isInt('' + id)) { @@ -22,32 +28,34 @@ function isAccountIdExist (id: number | string, res: Response) { promise = AccountModel.loadByUUID('' + id) } - return isAccountExist(promise, res) + return isAccountExist(promise, res, sendNotFound) } -function isLocalAccountNameExist (name: string, res: Response) { +function isLocalAccountNameExist (name: string, res: Response, sendNotFound = true) { const promise = AccountModel.loadLocalByName(name) - return isAccountExist(promise, res) + return isAccountExist(promise, res, sendNotFound) } -function isAccountNameWithHostExist (nameWithDomain: string, res: Response) { +function isAccountNameWithHostExist (nameWithDomain: string, res: Response, sendNotFound = true) { const [ accountName, host ] = nameWithDomain.split('@') let promise: Bluebird - if (!host) promise = AccountModel.loadLocalByName(accountName) - else promise = AccountModel.loadLocalByNameAndHost(accountName, host) + if (!host || host === CONFIG.WEBSERVER.HOST) promise = AccountModel.loadLocalByName(accountName) + else promise = AccountModel.loadByNameAndHost(accountName, host) - return isAccountExist(promise, res) + return isAccountExist(promise, res, sendNotFound) } -async function isAccountExist (p: Bluebird, res: Response) { +async function isAccountExist (p: Bluebird, res: Response, sendNotFound: boolean) { const account = await p if (!account) { - res.status(404) - .send({ error: 'Account not found' }) - .end() + if (sendNotFound === true) { + res.status(404) + .send({ error: 'Account not found' }) + .end() + } return false } @@ -60,6 +68,7 @@ async function isAccountExist (p: Bluebird, res: Response) { // --------------------------------------------------------------------------- export { + isAccountIdValid, isAccountIdExist, isLocalAccountNameExist, isAccountDescriptionValid,