Fix NoMethodError for BranchesController
What does this MR do and why?
Contributes to #460474 (closed)
Problem
BranchesController#index uses params
directly without verifying their
type.
Solution
Use StrongParameters to automatically exclude non string query parameters.
Screenshots or screen recordings
Screenshots are required for UI changes, and strongly recommended for all other merge requests.
Before | After |
---|---|
![]() |
![]() |
How to set up and validate locally
- Visit http://gdk.test:3000/gitlab-org/gitlab-test/-/branches/all?page[]=3
- You should not see a 500 error.
Edited by Vasilii Iakliushin