Project

General

Profile

Bug #5412

Webhook for "pullrequest commented" event returns incomplete data

Added by Rishipuri Bharatpuri almost 4 years ago. Updated over 3 years ago.

Status:
Resolved
Priority:
Normal
Assignee:
-
Category:
-
Target version:
-
Start date:
04.01.2018
Due date:
% Done:

0%

Estimated time:
Sorting:
Commit Number:
Affected Version:

Description

If there is a new pull request comment along with a status change, webhook returns the status as null instead of the actual status with the comment.

Here is the part of data returned:

   comment: {
      permalink_url: "https://rhodecode/_admin/pull-request/999#comment-6666",
      line: null,
      type: "note",
      status: null,
      text: "test test",
      file: null,
      url: "https://rhodecode/_admin/pull-request/999#comment-6666"
    },
#1

Updated by Rishipuri Bharatpuri almost 4 years ago

Is this an expected behaviour?

#2

Updated by Marcin Kuzminski [CTO] almost 4 years ago

Not expected, what version are you using?

#3

Updated by Rishipuri Bharatpuri almost 4 years ago

Currently on 4.10.6

#4

Updated by Marcin Kuzminski [CTO] almost 4 years ago

ok, we'll check that then.

#5

Updated by Marcin Kuzminski [CTO] almost 4 years ago

Hi,

We can confirm that. It's an issue related to database transaction isolation. We'll add a small fix to make sure the data is read properly in this case. The change is trivial, and we can send you a patch to fix this.

#6

Updated by Redmine Integration almost 4 years ago

  • Status changed from New to Resolved
#7

Updated by Rishipuri Bharatpuri over 3 years ago

Hi,

Was this fix released? If yes, which version of rhodecode has it?

#8

Updated by Marcin Kuzminski [CTO] over 3 years ago

Yes, 4.11 series has this fix.

#9

Updated by Rishipuri Bharatpuri over 3 years ago

Thank you.

Also available in: Atom PDF