diff options
| author | Ethan Lee <ethanalee@google.com> | 2026-04-06 21:37:29 +0000 |
|---|---|---|
| committer | Ethan Lee <ethanalee@google.com> | 2026-04-07 08:31:17 -0700 |
| commit | d1eedfe77fdd85e4ec9dcef7e4a4411116eddbb2 (patch) | |
| tree | 8180fa340a73de97dc6dd02ced56b1f4877f1490 | |
| parent | a82ef6233f8b2ebae8d4959c7c2596ec5e25dda2 (diff) | |
| download | go-x-pkgsite-d1eedfe77fdd85e4ec9dcef7e4a4411116eddbb2.tar.xz | |
internal/api: unify default pagination limits
Change-Id: Ie8faec1c02d74ac2559e78d8793d6050a98b38fc
Reviewed-on: https://go-review.googlesource.com/c/pkgsite/+/763281
Auto-Submit: Ethan Lee <ethanalee@google.com>
LUCI-TryBot-Result: Go LUCI <golang-scoped@luci-project-accounts.iam.gserviceaccount.com>
Reviewed-by: Jonathan Amsterdam <jba@google.com>
kokoro-CI: kokoro <noreply+kokoro@google.com>
| -rw-r--r-- | internal/api/api.go | 16 |
1 files changed, 8 insertions, 8 deletions
diff --git a/internal/api/api.go b/internal/api/api.go index f7ab8882..49873c9b 100644 --- a/internal/api/api.go +++ b/internal/api/api.go @@ -24,8 +24,8 @@ import ( const ( // maxSearchResults is the maximum number of search results to return for a search query. maxSearchResults = 1000 - // searchResultsPerPage is the number of search results to return per page for paginated search results. - searchResultsPerPage = 100 + // defaultLimit is the default number of results to return per page for paginated results. + defaultLimit = 100 ) // ServePackage handles requests for the v1 package metadata endpoint. @@ -150,7 +150,7 @@ func ServeModuleVersions(w http.ResponseWriter, r *http.Request, ds internal.Dat infos = filtered } - resp, err := paginate(infos, params.ListParams, 100) + resp, err := paginate(infos, params.ListParams, defaultLimit) if err != nil { return err } @@ -198,7 +198,7 @@ func ServeModulePackages(w http.ResponseWriter, r *http.Request, ds internal.Dat }) } - resp, err := paginate(items, params.ListParams, 100) + resp, err := paginate(items, params.ListParams, defaultLimit) if err != nil { return err } @@ -243,7 +243,7 @@ func ServeSearch(w http.ResponseWriter, r *http.Request, ds internal.DataSource) }) } - resp, err := paginate(results, params.ListParams, searchResultsPerPage) + resp, err := paginate(results, params.ListParams, defaultLimit) if err != nil { return fmt.Errorf("%w: %s", derrors.InvalidArgument, err.Error()) } @@ -291,7 +291,7 @@ func ServePackageSymbols(w http.ResponseWriter, r *http.Request, ds internal.Dat }) } - resp, err := paginate(items, params.ListParams, 100) + resp, err := paginate(items, params.ListParams, defaultLimit) if err != nil { return err } @@ -344,7 +344,7 @@ func ServePackageImportedBy(w http.ResponseWriter, r *http.Request, ds internal. importedBy = filtered } - paged, err := paginate(importedBy, params.ListParams, 100) + paged, err := paginate(importedBy, params.ListParams, defaultLimit) if err != nil { return err } @@ -401,7 +401,7 @@ func ServeVulnerabilities(vc *vuln.Client) func(w http.ResponseWriter, r *http.R }) } - resp, err := paginate(items, params.ListParams, 100) + resp, err := paginate(items, params.ListParams, defaultLimit) if err != nil { return err } |