Skip to content

bugfix(gitlab): Make owner and fork properties optional to fix GitLab error "Invalid arguments" #721

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom
Merged
olaservo merged 2 commits into from
Mar 29, 2025
Merged

bugfix(gitlab): Make owner and fork properties optional to fix GitLab error "Invalid arguments" #721

olaservo merged 2 commits into from
Mar 29, 2025

Conversation

@cong-toquang
Copy link
Contributor

@cong-toquang cong-toquang commented Mar 3, 2025 •
edited
Loading

Description

Server Details

  • Server: GitLab
  • Changes to: N/A

Motivation and Context

This PR fixes bug #357.

How Has This Been Tested?

I created a local MCP server and verified that the local GitLab MCP server is able to retrieve information from a self-hosted GitLab instance via Cline in VSCode.

Breaking Changes

None.

Types of changes

  • Bug fix (non-breaking change which fixes an issue)
  • New feature (non-breaking change which adds functionality)
  • Breaking change (fix or feature that would cause existing functionality to change)
  • Documentation update

Checklist

  • I have read the MCP Protocol Documentation
  • My changes follow MCP security best practices
  • I have updated the server's README accordingly
  • I have tested this with an LLM client
  • My code follows the repository's style guidelines
  • New and existing tests pass locally
  • I have added appropriate error handling
  • I have documented all environment variables and configuration options

Additional context

@cong-toquang cong-toquang changed the title feat(gitlab): Make owner and fork properties optional feat(gitlab): Make owner and fork properties optional (fix bug GitLab Invalid arguments #357) Mar 3, 2025
@cong-toquang cong-toquang changed the title feat(gitlab): Make owner and fork properties optional (fix bug GitLab Invalid arguments #357) feat(gitlab): Make owner and fork properties optional to fix GitLab error "Invalid arguments" Mar 3, 2025
@cong-toquang cong-toquang changed the title feat(gitlab): Make owner and fork properties optional to fix GitLab error "Invalid arguments" bugfix(gitlab): Make owner and fork properties optional to fix GitLab error "Invalid arguments" Mar 4, 2025
@francovp francovp mentioned this pull request Mar 14, 2025
Closed
@olaservo olaservo added server-gitlab bug Something isn't working labels Mar 27, 2025
olaservo
olaservo approved these changes Mar 29, 2025
View reviewed changes
Copy link
Member

@olaservo olaservo left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks for fixing this and for your patience in this getting merged.

@olaservo olaservo merged commit 507f524 into modelcontextprotocol:main Mar 29, 2025
25 checks passed
This was referenced Mar 29, 2025
Closed
Closed
PazerOP referenced this pull request in PazerOP/mcp-template Jul 15, 2025
@olaservo
Merge pull request #721 from cong-toquang/main
2506301 
bugfix(gitlab): Make owner and fork properties optional to fix GitLab error "Invalid arguments"
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@olaservo olaservo olaservo approved these changes

Assignees

No one assigned

Labels

bug Something isn't working

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

2 participants

@cong-toquang @olaservo