Uploaded image for project: 'Bitbucket Data Center'
  1. Bitbucket Data Center
  2. BSERV-13549

Requesting required reviewers allows commit hashes in the sourceRefId field

    XMLWordPrintable

Details

    • Bug
    • Resolution: Unresolved
    • Low
    • None
    • 8.0.5
    • API - REST

    Description

      Issue Summary

      This is reproducible on Data Center: Yes

      The API docs state that a SourceRefID needs to be specified in the required reviewers request. A valid RefID is considered a branch or tag name, but it is possible to put a commit hash in the SourceRefID field. Only branch or tags should be allowed. 

      If there are 2 source branches with the same commit hash, it's not clear which branch the required reviewers will be obtained from. 

      Steps to Reproduce

       

      Perform a call to endpoint,

      rest/default-reviewers/1.0/projects/{projectKey}/repos/{repositorySlug}/reviewers

       Set SourceRefID set to the commit hash of the head of the source branch.

      http://<bitbucket>:7990/rest/default-reviewers/1.0/projects/{projectKey}/repos/{repositorySlug}/reviewers?sourceRepoId=sourceRepoId&targetRepoId=targetRepoId&sourceRefId=6b06eb0f9b8&targetRefId=branchName

      Expected Results

      An error showing the RefId cannot be found in the repo : 

      http://centos001:7990/rest/default-reviewers/1.0/projects/\{projectKey}/repos/{repositorySlug}/reviewers?sourceRepoId=2&targetRepoId=2&sourceRefId=6b06eb0f9b8&targetRefId=branchName (404)cache-control: no-cache, no-transform
      connection: keep-alive
      content-encoding: gzip
      content-type: application/json;charset=UTF-8
      date: Thu, 14 Jul 2022 00:38:15 GMT
      keep-alive: timeout=60
      transfer-encoding: chunked
      vary: x-ausername,x-auserid,cookie,accept-encoding
      x-arequestid: @1AXD1TJx1298x110x0
      x-asessionid: 100w2gg
      x-auserid: 52
      x-ausername: Admin
      x-content-type-options: nosniff
      { "errors": [
      { "context": null, "message": "Repository TES/pssrv-42288 doesn't have a 6b06eb0f9b8 ref.", "exceptionName": "com.atlassian.bitbucket.repository.NoSuchBranchException" }
      ]
      }
      

       

      Actual Results

      The reviewers are returned. 

      http://ubuntu005:7990/rest/default-reviewers/1.0/projects/{projectKey}/repos/{repositorySlug}/reviewers?sourceRepoId=2&targetRepoId=2&sourceRefId=6b06eb0f9b8&targetRefId=branchName (200)
      cache-control: no-cache, no-transform
      connection: keep-alive
      content-encoding: gzip
      content-type: application/json
      date: Thu, 14 Jul 2022 00:02:01 GMT
      keep-alive: timeout=60
      transfer-encoding: chunked
      vary: x-ausername,x-auserid,cookie,accept-encoding
      x-arequestid: @BHLZU2x2x2706x0
      x-asessionid: 1k8ng6i
      x-auserid: 2
      x-ausername: Admin
      x-content-type-options: nosniff
      
      [
        {
          "name": "Reviewer",
          "emailAddress": "Reviewer@test.test",
          "id": 4,
          "displayName": "Reviewer",
          "active": true,
          "slug": "reviewer",
          "type": "NORMAL",
          "links": {
            "self": [
              {
                "href": "http://ubuntu005:7990/users/reviewer"
              }
            ]
          }
        }
      ] 

      Workaround

      N/A

      Attachments

        Issue Links

          Activity

            People

              Unassigned Unassigned
              e74118b4e28d David Jansons
              Votes:
              0 Vote for this issue
              Watchers:
              1 Start watching this issue

              Dates

                Created:
                Updated: