diff options
| author | Armin Hohenegger <armin695@gmail.com> | 2019-08-10 19:11:20 +0200 |
|---|---|---|
| committer | Armin Hohenegger <armin695@gmail.com> | 2019-08-10 19:11:20 +0200 |
| commit | 20ac5e6d4b1e1b6807e13b11daaedd2fb47bd3fc (patch) | |
| tree | 3e1fc3a651668becaa3961c6ad98f97b7b57d676 /changelogs | |
| parent | ea671dec7f9db06c1d45ddd1536c1af194de8f62 (diff) | |
| download | gitlab-ce-20ac5e6d4b1e1b6807e13b11daaedd2fb47bd3fc.tar.gz | |
fix handling of empty ref_name parameter string in commits api
when params[:ref_name] is set to "" by passing an empty query parameter
to the api it is evaluated as false by the || operator.
The use of active support core extensions presence method fixes the original
implemantation.
https://guides.rubyonrails.org/active_support_core_extensions.html#presence
Diffstat (limited to 'changelogs')
| -rw-r--r-- | changelogs/unreleased/fix-commits-api-empty-refname.yml | 5 |
1 files changed, 5 insertions, 0 deletions
diff --git a/changelogs/unreleased/fix-commits-api-empty-refname.yml b/changelogs/unreleased/fix-commits-api-empty-refname.yml new file mode 100644 index 00000000000..efdb950e45d --- /dev/null +++ b/changelogs/unreleased/fix-commits-api-empty-refname.yml @@ -0,0 +1,5 @@ +--- +title: Fix 500 errors in commits api caused by empty ref_name parameter +merge_request: +author: +type: fixed |