summaryrefslogtreecommitdiff
path: root/src/api/endpoints/posts
diff options
context:
space:
mode:
Diffstat (limited to 'src/api/endpoints/posts')
-rw-r--r--src/api/endpoints/posts/create.js40
1 files changed, 13 insertions, 27 deletions
diff --git a/src/api/endpoints/posts/create.js b/src/api/endpoints/posts/create.js
index 57e95bd712..2f03ebd8e9 100644
--- a/src/api/endpoints/posts/create.js
+++ b/src/api/endpoints/posts/create.js
@@ -4,8 +4,9 @@
* Module dependencies
*/
import * as mongo from 'mongodb';
+import validate from '../../validator';
import parse from '../../../common/text';
-import Post from '../../models/post';
+import { Post, isValidText } from '../../models/post';
import User from '../../models/user';
import Following from '../../models/following';
import DriveFile from '../../models/drive-file';
@@ -16,15 +17,14 @@ import event from '../../event';
import config from '../../../conf';
/**
- * 最大文字数
- */
-const maxTextLength = 1000;
-
-/**
* 添付できるファイルの数
*/
const maxMediaCount = 4;
+function hasDuplicates(array) {
+ return (new Set(array)).size !== array.length;
+}
+
/**
* Create a post
*
@@ -37,30 +37,16 @@ module.exports = (params, user, app) =>
new Promise(async (res, rej) =>
{
// Get 'text' parameter
- let text = params.text;
- if (text !== undefined && text !== null) {
- if (typeof text != 'string') {
- return rej('text must be a string');
- }
- text = text.trim();
- if (text.length == 0) {
- text = null;
- } else if (text.length > maxTextLength) {
- return rej('too long text');
- }
- } else {
- text = null;
- }
+ const [text, textErr] = validate(params.text, 'string', false, isValidText);
+ if (textErr) return rej('invalid text');
// Get 'media_ids' parameter
- let medias = params.media_ids;
- let files = [];
- if (medias !== undefined && medias !== null) {
- if (!Array.isArray(medias)) {
- return rej('media_ids must be an array');
- }
+ const [mediaIds, mediaIdsErr] = validate(params.media_ids, 'array', false, x => !hasDuplicates(x));
+ if (mediaIdsErr) return rej('invalid media_ids');
- if (medias.length > maxMediaCount) {
+ let files = [];
+ if (mediaIds !== null) {
+ if (mediaIds.length > maxMediaCount) {
return rej('too many media');
}