Kaggle / kaggle-api

Official Kaggle API
Apache License 2.0
6.01k stars 1.06k forks source link

Listing datasets allows 0 for `page` query parameter #593

Closed anorderh closed 1 week ago

anorderh commented 1 week ago

When making HTTP requests to the following API endpoint

https://www.kaggle.com/api/v1/datasets/list?page=0

I am able to receive valid responses. This seems incorrect when referencing the source code

if int(page) <= 0:
            raise ValueError('Page number must be >= 1')

My testing did not include using the CLI tool, so this may only be a server issue.

stevemessick commented 1 week ago

Thanks for the report. Since we do not officially support this functionality, I'm closing this issue.