Skip to content

fix: allow nested GitLab repository groups#1365

Open
go165 wants to merge 1 commit into
modelcontextprotocol:mainfrom
go165:go165/gitlab-nested-group-urls
Open

fix: allow nested GitLab repository groups#1365
go165 wants to merge 1 commit into
modelcontextprotocol:mainfrom
go165:go165/gitlab-nested-group-urls

Conversation

@go165

@go165 go165 commented Jun 13, 2026

Copy link
Copy Markdown

Summary

  • Allow GitLab repository URLs with nested group/subgroup paths.
  • Keep each GitLab path segment non-empty so malformed URLs with empty segments are still rejected.
  • Add validator coverage for a GitLab repository under nested subgroups.

Fixes #1359.

Verification

  • gofmt -w internal/validators/utils.go internal/validators/validators_test.go
  • git diff --check
  • Attempted go test ./internal/validators, but local verification is blocked by the Go 1.26 toolchain requirement: go: download go1.26 for linux/amd64: toolchain not available

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

GitLab repository URLs with nested groups/subgroups are rejected by validator

1 participant