GET /collections ignores the documented offset parameter
まだ誰も着手していません。
評価
- 難易度
- 3/5
- 見積もり時間
- 1〜2日
- 初心者へのやさしさ
- 74/100
- issue の種類
- バグ
- 明瞭さ
- 明確に書かれている
- 活発さ
- 活発
- 技術スタック
- javascript
調査の方向性
Start with app/controllers/collections-controller.js and app/repository/collections-repository.js, where the issue identifies the offset/skip mismatch. Add regression tests comparing paged collection identities with an unpaged baseline for nonzero and beyond-end offsets, unlimited results, and latest/all revision selection; done means the pages return distinct collections and the specified offset behavior holds when limit is 0.
索引モデルが issue の本文から書いたものです。
説明
Verified on REST API 4.24.0, commit 69b37fe769e6abe7675d413451a8361aecc36bea.
Reproduction and impact
Create three valid collections matching a unique search term. Request:
GET /api/collections?search=UNIQUE_TERM&limit=1&offset=0
GET /api/collections?search=UNIQUE_TERM&limit=1&offset=1
GET /api/collections?search=UNIQUE_TERM&limit=1&offset=2
GET /api/collections?search=UNIQUE_TERM&limit=1&offset=3
Every request returns the same first collection. Unpaged baselines before and after match. limit=0&offset=1 also returns all three instead of the final two. Collection GET returns an array; no includePagination parameter is needed.
This reproduces through direct HTTP requests. Repeated pages prevent traversal of the matching collections.
Root cause and expected fix
The controller supplies options.offset, but the repository checks options.skip. Nothing translates between them.
Use the documented offset after filtering/version grouping and before limiting. Offsets 0–2 should return distinct collections; offset 3 should return []. Apply offset even when limit=0.
Add regression tests comparing page identities with an unpaged baseline, including nonzero/beyond-end offsets, unlimited results, and latest/all revision selection. This is separate from collection creation #505 and index refresh #506.
- 主要言語
- JavaScript
- スター
- 57
- フォーク
- 18
- 平均マージ
- 1日 26分
- マージ済み PR（30日）
- 7
環境構築
- Dockerfile または Docker Compose ファイルあり
- プルリクエストのテンプレートなし
- コントリビューションガイドを読む
はじめの一歩
- issue を最後まで読み、次にプロジェクトのコントリビューションガイドを読みます。
- 着手することを issue にコメントします — 二人が同じ作業をするのを防げます。
- リポジトリをフォークし、ブランチを切って変更します。
- issue 番号を参照したプルリクエストを送ります。
mitre-attack/attack-workbench-rest-api のほかの issue
-
難易度 2/5 1〜3時間 初心者へのやさしさ 85/100
-
難易度 2/5 1〜3時間 初心者へのやさしさ 86/100
-
難易度 2/5 1〜3時間 初心者へのやさしさ 86/100
-
難易度 3/5 1〜2日 初心者へのやさしさ 68/100
-
難易度 3/5 1〜2日 初心者へのやさしさ 68/100
mitre-attack/attack-workbench-rest-api の issue をすべて見る
似ている issue
-
難易度 2/5 1〜3時間 初心者へのやさしさ 66/100
メンテナーはふだん 3 日以内に返信
-
audit.md numbers Theming and Responsive Design differently in the headings and the score table対応中かも @pbakaus が今日担当しました。 オープンneeds triage
難易度 1/5 1時間未満 初心者へのやさしさ 90/100
pbakaus/impeccable#979 ·
メンテナーはふだん 1 日以内に返信
-
area/web interface
難易度 2/5 1〜3時間 初心者へのやさしさ 78/100
mastodon/mastodon#40924 · コメント 1 件 ·
メンテナーはふだん 1 日以内に返信
-
fireEvent.select does not wrap its automatic native focus in act対応中かも @sergioperezcheco が今日担当しました。 オープン
難易度 2/5 1〜3時間 初心者へのやさしさ 88/100
-
bug user-priority/P2
難易度 1/5 1時間未満 初心者へのやさしさ 92/100
メンテナーはふだん 1 日以内に返信