Skip to content
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.

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

Adding a reviewer to a GitLab merge request is not working #4385

Closed
ctosin opened this issue Nov 8, 2021 · 4 comments · Fixed by #4747
Closed

Adding a reviewer to a GitLab merge request is not working #4385

ctosin opened this issue Nov 8, 2021 · 4 comments · Fixed by #4747
Labels
T: bug 🐞 Something isn't working V: GitLab Issues relating to GitLab support

Comments

@ctosin
Copy link

ctosin commented Nov 8, 2021

The functionality below, which adds reviewers to a GitLab merge request, does not seem to be working properly.

def add_approvers_to_merge_request(merge_request)

It is trying to call a URL that does not exist, therefore a 404 is being returned.

Actually the problem seems to be in the GitLab dependency that Dependabot is using. This is the invalid URL: /projects/#{url_encode project}/merge_requests/#{merge_request}/approvers.

@ctosin ctosin added the T: bug 🐞 Something isn't working label Nov 8, 2021
@jurre
Copy link
Member

jurre commented Nov 8, 2021

We rely on community contributions for GitLab support currently, I'd be happy to accept a PR that fixes this, but am wondering if this could be some sort of breaking change in the gitlab url format?

@andrcuns
Copy link
Contributor

andrcuns commented Nov 10, 2021

This is a know issue, the endpoint was removed starting from gitlab version 14.

Unfortunately https://github.com/narkoz/gitlab is taking extremely long to release a new version where the endpoint is updated in master.

Once it is released, I could prepare a PR that fixes it, I don't think dependabot-core project would be ok to build the gitlab gem from the master branch.

@brrygrdn brrygrdn added the V: GitLab Issues relating to GitLab support label Nov 26, 2021
@floh96
Copy link

floh96 commented Jan 20, 2022

@andrcuns there was a new release of the gitlab gem. Is it now possible to fix this issue? thanks

@andrcuns
Copy link
Contributor

I believe it is, yes. I might take a look when I have some free time.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
T: bug 🐞 Something isn't working V: GitLab Issues relating to GitLab support
Projects
None yet
Development

Successfully merging a pull request may close this issue.

5 participants