Skip to content

Handle missing .gitmodules when getting submodule urls

Jacob Vosmaer (GitLab) requested to merge malformed-submodule into master

What does this MR do?

Adds a test for a submodule parsing edge case and fixes a related bug.

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

Why was this MR needed?

This would throw an exception if there was no .gitmodules.

Screenshots (if relevant)

Does this MR meet the acceptance criteria?

What are the relevant issue numbers?

https://gitlab.com/gitlab-org/gitaly/issues/367

https://sentry.gitlap.com/gitlab/gitlabcom/issues/40783/

Merge request reports