Skip to content
Snippets Groups Projects

feature and fix for 500 on group missing

Merged James Lopez requested to merge jameslopez/gitlab-ce:fix/unexistent-group-500 into master

Fixes https://gitlab.com/gitlab-org/gitlab-ce/issues/3352

And maybe https://gitlab.com/gitlab-org/gitlab-ce/issues/5600 (looking into this one now, as it may require a different fix).

With the fix it will return a 404 if a group is not found. Added relevant feature.

Merge request reports

Loading
Loading

Activity

Filter activity
  • Approvals
  • Assignees & reviewers
  • Comments (from bots)
  • Comments (from users)
  • Commits & branches
  • Edits
  • Labels
  • Lock status
  • Mentions
  • Merge request status
  • Tracking
  • James Lopez Added 1 commit:

    Added 1 commit:

    • eef9c9d4 - update fix based on feedback
  • James Lopez Added 1 commit:

    Added 1 commit:

    • 5eb936dc - moved and refactored feature based on feedback. Also updated CHANGELOG
  • James Lopez Added 43 commits:

    Added 43 commits:

    • 5eb936dc...5e0ee54c - 42 commits from branch gitlab-org:master
    • afde800b - Merge branch 'master' of gitlab.com:gitlab-org/gitlab-ce into fix/unexistent-group-500
  • Robert Speicher Status changed to merged

    Status changed to merged

  • Robert Speicher mentioned in commit 56538f9c

    mentioned in commit 56538f9c

  • @jameslopez I marked it as fixing both of the mentioned issues. One was already closed, however.

  • Please register or sign in to reply
    Loading