Skip to content

Fix searching for files by path

Nick Thomas requested to merge (removed):36939-fix-find-blobs-by-path into master

What does this MR do?

Modifies Gitlab::FileFinder to return Gitlab::SearchResult::FoundBlob results for blobs found by filename

Are there points in the code the reviewer needs to double check?

We're pulling the actual blob contents from disc where previously we wouldn't. I'm using a batch call to do so, and limiting the size of the returned data, but do I need to be more restrictive here?

Why was this MR needed?

Searching is broken in some cases

Screenshots (if relevant)

Does this MR meet the acceptance criteria?

What are the relevant issue numbers?

Closes #36939 (closed)

Edited by Nick Thomas

Merge request reports

Loading