Skip to content
Snippets Groups Projects
Commit b0fa5c3a authored by Dean's avatar Dean
Browse files

Merge branch 'master' into 'master'

Set `maxLimit` as default value for the `limit` parameter in the `/objects` endpoint.

See merge request !3
parents 4dd6ce3f ca8536cf
No related branches found
No related tags found
1 merge request!3Set maxLimit as default value for limit param
......@@ -49,11 +49,16 @@ func ListObjects(w http.ResponseWriter, r *http.Request) {
}
// Determine offset, limit and filter params
query := r.URL.Query()
l := query.Get("limit")
limit, err := strconv.Atoi(l)
if err != nil || limit < 0 {
panic(apierrors.InvalidOffsetOrLimit)
var (
query = r.URL.Query()
limit = maxLimit
l = query.Get("limit")
)
if l != "" {
limit, err := strconv.Atoi(l)
if err != nil || limit < 0 {
panic(apierrors.InvalidOffsetOrLimit)
}
}
if limit > maxLimit {
panic(apierrors.LimitTooLarge)
......
0% Loading or .
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment