Resolve "Web IDE crashes for directories named "blob"" (!20712) · Merge Requests · GitLab.org / GitLab FOSS

Resolve "Web IDE crashes for directories named "blob""

What does this MR do?

Fixes the bug where the Web IDE crashes when files match edit|tree|blob

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

(See discussion below)

Why was this MR needed?

#48036 (closed)

Screenshots (if relevant)

Does this MR meet the acceptance criteria?

Changelog entry added, if necessary
~~[ ] Documentation created/updated~~
~~[ ] API support added~~
Tests added for this feature/bug
Conforms to the code review guidelines
- ~~[ ] Has been reviewed by a UX Designer~~
- Has been reviewed by a Frontend maintainer
- ~~[ ] Has been reviewed by a Backend maintainer~~
- ~~[ ] Has been reviewed by a Database specialist~~
Conforms to the merge request performance guidelines
Conforms to the style guides
Conforms to the database guides
If you have multiple commits, please combine them into a few logically organized commits by squashing them
Internationalization required/considered
End-to-end tests pass (package-and-qa manual pipeline job)

What are the relevant issue numbers?

Closes #48036 (closed)

## What does this MR do?

Fixes the bug where the Web IDE crashes when files match `edit|tree|blob`

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

(See discussion below)

## Why was this MR needed?

#48036

## Screenshots (if relevant)

![20180719_webide_blob_fix](/uploads/f452dca19931b3dae5606cc2211103c9/20180719_webide_blob_fix.gif)

## Does this MR meet the acceptance criteria?

- [ ] [Changelog entry](https://docs.gitlab.com/ee/development/changelog.html) added, if necessary
- ~~[ ] [Documentation created/updated](https://docs.gitlab.com/ee/development/documentation/index.html#contributing-to-docs)~~
- ~~[ ] [API support added](https://docs.gitlab.com/ee/development/api_styleguide.html)~~
- [ ] [Tests added for this feature/bug](https://docs.gitlab.com/ee/development/testing_guide/index.html)
- Conforms to the [code review guidelines](https://docs.gitlab.com/ee/development/code_review.html)
  - ~~[ ] Has been reviewed by a UX Designer~~
  - [ ] Has been reviewed by a Frontend [maintainer](https://about.gitlab.com/handbook/engineering/#maintainer)
  - ~~[ ] Has been reviewed by a Backend [maintainer](https://about.gitlab.com/handbook/engineering/#maintainer)~~
  - ~~[ ] Has been reviewed by a Database [specialist](https://about.gitlab.com/team/structure/#specialist)~~
- [ ] Conforms to the [merge request performance guidelines](https://docs.gitlab.com/ee/development/merge_request_performance_guidelines.html)
- [ ] Conforms to the [style guides](https://gitlab.com/gitlab-org/gitlab-ee/blob/master/CONTRIBUTING.md#style-guides)
- [ ] Conforms to the [database guides](https://docs.gitlab.com/ee/development/README.html#databases-guides)
- [ ] If you have multiple commits, please combine them into a few logically organized commits by [squashing them](https://git-scm.com/book/en/Git-Tools-Rewriting-History#Squashing-Commits)
- [ ] [Internationalization required/considered](https://docs.gitlab.com/ee/development/i18n/index.html)
- [ ] [End-to-end tests](https://docs.gitlab.com/ee/development/testing_guide/end_to_end_tests.html#testing-code-in-merge-requests) pass (`package-and-qa` manual pipeline job)

## What are the relevant issue numbers?

Closes #48036

Edited Jul 19, 2018 by Paul Slaughter