fix: 修复 list_dir 函数在分页查询参数错误的问题 (TencentBlueKing#235)

Author: SheepSheepChen

sdks/bk-storages/CHANGE.md

@@ -1,5 +1,9 @@
 ## Change logs
 
+### 2.0.1
+
+- 修复 `BKRepoStorage` list_dir 函数在分页查询参数错误的问题
+
 ### 2.0.0
 
 - 支持 Python 3.11，移除对 Python 3.6, 3.7 的支持

sdks/bk-storages/bkstorages/__init__.py

@@ -8,4 +8,4 @@
  * an "AS IS" BASIS, WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied. See the License for the
  * specific language governing permissions and limitations under the License.
 """
-__version__ = "2.0.0"
+__version__ = "2.0.1"

sdks/bk-storages/bkstorages/backends/bkrepo.py

@@ -236,7 +236,7 @@ def __list_dir(self, key_prefix: str, cur_page: int = 1) -> Tuple[List, List, bo
         client = self.get_client()
         url = urljoin(self.endpoint_url, f"/repository/api/node/page/{self.project}/{self.bucket}/{key_prefix}")
         # NOTE: 按分页查询 bkrepo 的文件数, 1000 是一个经验值, 设置仅可能大的数值是避免发送太多次请求到 bk-repo
-        params = {"pageSize": 1000, "PageNumber": cur_page, "includeFolder": True}
+        params = {"pageSize": 1000, "pageNumber": cur_page, "includeFolder": True}
         resp = client.get(url, params=params, timeout=TIMEOUT_THRESHOLD)
         data = self._validate_resp(resp)
         total_pages = data["totalPages"]

sdks/bk-storages/pyproject.toml

@@ -1,6 +1,6 @@
 [tool.poetry]
 name = "bkstorages"
-version = "2.0.0"
+version = "2.0.1"
 description = "File storage backends for blueking PaaS platform"
 readme = "README.md"
 authors = ["blueking <blueking@tencent.com>"]

sdks/bk-storages/tests/test_bkrepo.py

@@ -134,7 +134,7 @@ def test_list_dir(self, bk_repo_storage, adapter, endpoint, mock_responses, expe
                 urljoin(
                     endpoint,
                     f"/repository/api/node/page/dummy-project/dummy-bucket/"
-                    f"{path}?pageSize=1000&PageNumber={idx + 1}&includeFolder=True",
+                    f"{path}?pageSize=1000&pageNumber={idx + 1}&includeFolder=True",
                 ),
                 json={"code": 0, "message": 0, "data": {"totalPages": len(mock_responses), "records": records}},
             )