diff options
author | Alec Warner <antarus@gentoo.org> | 2020-05-26 12:50:50 -0700 |
---|---|---|
committer | Alec Warner <antarus@gentoo.org> | 2020-05-26 12:51:26 -0700 |
commit | c9ae2b98d99fed6e35a7e65670ecb57c2b48da58 (patch) | |
tree | c9fc807de3f9e2b34b16e7da7d1c56e05166288e /pkg/app/handler/packages/utils.go | |
parent | Don't throw a 500 when users search with no query. (diff) | |
download | soko-c9ae2b98d99fed6e35a7e65670ecb57c2b48da58.tar.gz soko-c9ae2b98d99fed6e35a7e65670ecb57c2b48da58.tar.bz2 soko-c9ae2b98d99fed6e35a7e65670ecb57c2b48da58.zip |
Handle the case where a query param is missing.
Signed-off-by: Alec Warner <antarus@gentoo.org>
Diffstat (limited to 'pkg/app/handler/packages/utils.go')
-rw-r--r-- | pkg/app/handler/packages/utils.go | 8 |
1 files changed, 4 insertions, 4 deletions
diff --git a/pkg/app/handler/packages/utils.go b/pkg/app/handler/packages/utils.go index 13958c2..454da8a 100644 --- a/pkg/app/handler/packages/utils.go +++ b/pkg/app/handler/packages/utils.go @@ -239,9 +239,9 @@ func getReverse(index int, versions []*models.Version) *models.Version { // getParameterValue returns the value of a given parameter func getParameterValue(parameterName string, r *http.Request) string { - results, _ := r.URL.Query()[parameterName] - param := results[0] - return param + results, ok := r.URL.Query()[parameterName]; if !ok { return "" } + if len(results) == 0 { return "" } + return results[0] } // getPackageUseflags retrieves all local USE flags, global USE @@ -394,4 +394,4 @@ func containsUseflag(useflag models.Useflag, useflags []models.Useflag) bool { } } return false -}
\ No newline at end of file +} |