Skip to content

GitLab

  • Projects
  • Groups
  • Snippets
  • Help
    • Loading...
  • Help
    • Help
    • Support
    • Community forum
    • Submit feedback
    • Contribute to GitLab
  • Sign in
go
go
  • Project overview
    • Project overview
    • Details
    • Activity
    • Releases
  • Repository
    • Repository
    • Files
    • Commits
    • Branches
    • Tags
    • Contributors
    • Graph
    • Compare
  • Issues 21
    • Issues 21
    • List
    • Boards
    • Labels
    • Service Desk
    • Milestones
  • Merge Requests 0
    • Merge Requests 0
  • CI / CD
    • CI / CD
    • Pipelines
    • Jobs
    • Schedules
  • Operations
    • Operations
    • Incidents
    • Environments
  • Packages & Registries
    • Packages & Registries
    • Container Registry
  • Analytics
    • Analytics
    • CI / CD
    • Repository
    • Value Stream
  • Wiki
    • Wiki
  • Snippets
    • Snippets
  • Members
    • Members
  • Collapse sidebar
  • Activity
  • Graph
  • Create a new issue
  • Jobs
  • Commits
  • Issue Boards
  • SRCT
  • gogo
  • Merge Requests
  • !85

Merged
Opened Apr 18, 2017 by Nicholas J Anderson@nander13Owner

Fix collision resolution

  • Overview 0
  • Commits 1
  • Pipelines 1
  • Changes 1

Previously, this would just exit if a collision was found since it would recheck the same short url over and over again.

Additionally, it would not increment the cache counter on successful short generation since the error would cause the rest of the stanza to no longer execute

Assignee
Assign to
Reviewer
Request review from
Go 2.2 - Bloomin' Onion Edition
Milestone
Go 2.2 - Bloomin' Onion Edition (Past due)
Assign milestone
Time tracking
Reference: srct/go!85
Source branch: nanderson-fixcollision

Revert this merge request

This will create a new commit in order to revert the existing changes.

Switch branch
Cancel
A new branch will be created in your fork and a new merge request will be started.

Cherry-pick this merge request

Switch branch
Cancel
A new branch will be created in your fork and a new merge request will be started.