Skip to content

Conversation

@jmini
Copy link
Contributor

@jmini jmini commented May 24, 2023

Fixes #320

Testing done

Submitter checklist

  • Make sure you are opening from a topic/feature/bugfix branch (right side) and not your main branch!
  • Ensure that the pull request title represents the desired changelog entry
  • Please describe what you did
  • Link to relevant issues in GitHub or Jira
  • Link to relevant pull requests, esp. upstream and downstream changes
  • Ensure you have provided tests - that demonstrates feature works or fixes the issue

@jmini
Copy link
Contributor Author

jmini commented May 24, 2023

I will test it on my jenkins test installation and add some report here.

@jetersen
Copy link
Member

Would be nice to find the changelog for GitLab t document the supported GitLab version.

@jmini
Copy link
Contributor Author

jmini commented May 24, 2023

The issue linked by @ChristopherZellermann indicates that the hyphen was introduced 3 years ago:
https://gitlab.com/gitlab-org/gitlab/-/issues/214217

But now with GitLab 16.0, it seems that the URL without the hyphen are no longer working.

Even more details from:
https://gitlab.com/gitlab-org/gitlab/-/merge_requests/112075/diffs#61f0ea3834ef63ce9a722914025e075aad9b3f2d

When subgroups were introduced in GitLab 9.0, a /-/ delimiter was added to URLs to signify the end of a group path. All GitLab URLs now use this delimiter for project, group, and instance level features.

GitLab 16.0 removes legacy URLs from the GitLab application.

(this is probably also published somewhere on the website)


I just tested with our 15.x installation, and there the redirect was working:
<server>/a-group/a-subgroup/a-poject/merge_requests/135

was redirecting to:
<server>/a-group/a-subgroup/a-poject/-/merge_requests/135


So I think it is time to adjust this plugin.

@jetersen
Copy link
Member

Thanks for info. Thats helps make this easy change even easier :)

@jetersen jetersen added the bug Something isn't working label May 24, 2023
@jetersen jetersen merged commit d45c0f4 into jenkinsci:master May 24, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

bug Something isn't working

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Generated URLs for MR, commits, tags, etc. no longer working

2 participants