Allow Zoekt in Search API for blobs scope when Elasticsearch disabled
What does this MR do and why?
-
Before, we were checking for
use_elasticsearch?only. Zoekt was not getting considered, and it was returned asbad_request. Updated the logic to usesearch_typethat considerszoektalso. -
Some Rubocop fixes
References
Screenshots or screen recordings
| Before | After |
|---|---|
![]() |
![]() |
How to set up and validate locally
- Disable Advanced search from Application settings and keep Zoekt search enabled.
- Send an API request like this: https://gdk.test:3443/api/v4/groups/24/search?scope=blobs&search=test&search_type=zoekt.
- On master it was error which is wrong.
- On the MR branch, it is returning results.
MR acceptance checklist
Evaluate this MR against the MR acceptance checklist. It helps you analyze changes to reduce risks in quality, performance, reliability, security, and maintainability.
Edited by Ravi Kumar

