fix: Add validators to lsx API #9182
Conversation
|
yuki-takei
changed the title
|
|
||
| import type { PageQuery, PageQueryBuilder } from './generate-base-query'; | ||
|
|
||
| import { listPages } from '.'; | ||
|
|
||
| interface listPagesRequest extends Request<undefined, undefined, undefined, LsxApiParams> { |
There was a problem hiding this comment.
インターフェース名を IListPagesRequest にしてください
| pagePath: removeTrailingSlash(req.query.pagePath), | ||
| offset: req.query?.offset != null ? req.query.offset : undefined, | ||
| limit: req.query?.limit != null ? req.query.limit : undefined, | ||
| options: req.query?.options != null ? req.query.options : {}, |
There was a problem hiding this comment.
query の型を LsxApiParams 決め打ちにしているけれど、express-validator でチェックしておかないと予期しない値(例えば string[] など)が入ってきた時のエラーハンドリングが弱いんじゃないかな
(元々の実装も弱いんですがそれは置いておいて…)
There was a problem hiding this comment.
list-pages/index.ts 内ではなく server/index.ts で validationResult を使用する形でチェックを実装しました。
タスク
概要
変更点
セルフチェック