PUSH event hooks not sent for branch removal

[stephenc]

• Gitea version (or commit ref): 858197b
• Git version: N/A
• Operating system: N/A
• Database (use [x]): N/A
  • PostgreSQL
  • MySQL
  • MSSQL
  • SQLite
• Can you reproduce the bug at https://try.gitea.io:
  • Yes (provide example URL)
  • No
  • Not relevant
• Log gist:

Description

When I configure a webhook that receives all events, push events that remove a branch or branch removal of the branch through the UI do not trigger a webhook event:

Creation of a new branch results in events like:

X-Gitea-Event: create

{
  "secret": "magic",
  "sha": "2409990415d77f47ed5c4c775abda4ffa6aa5f11",
  "ref": "branch",
  "ref_type": "branch",
  "repository": {
    "id": 17,
    "owner": {
...

and

X-Gitea-Event: push

{
  "secret": "magic",
  "ref": "refs/heads/branch",
  "before": "0000000000000000000000000000000000000000",
  "after": "2409990415d77f47ed5c4c775abda4ffa6aa5f11",
...

I would expect removal of the branch to trigger at least a webhook that looked something like:

X-Gitea-Event: push

{
  "secret": "magic",
  "ref": "refs/heads/branch",
  "before": "2409990415d77f47ed5c4c775abda4ffa6aa5f11",
  "after": "0000000000000000000000000000000000000000",
...

It is a somewhat separate concern the use of a magic value to indicate no ref, better would be to include the created and deleted booleans that GitHub does, but for my part the only concern I have is the lack of a push event to catch branch removal.

[bkcsoft]

Could you include the logs from gitea? without them it's very hard to see what's happening :)

[bkcsoft]

(Not sure if this is bug or not, might be a feature request 😅 )

[stephenc]

I'll see if I can dig it out but your Jenkins integration (which I have nearly finished) will be annoying without an event for this.

If you want feature requests:

• manage hooks for a user's repos
• events for repo created / deleted
• some way in the api to distinguish a name as an org name vs user name (currently I get the name as a user and it's and org if the email is empty)
• file browse by rest api (basically the raw needs a way to serve directory listings for non-file paths... also good to flag content as a link if somebody commits a link)

Icing on top:

• commit file (with branch creation if desired) by rest api - needed for blue ocean integration

[stephenc]

Logs

Creating the branch: (git push origin branch)

Jul  3 20:05:56 sshd[27634]: Accepted publickey for git from x.x.x.x port 58096 ssh2: RSA SHA256:BTv+kWSXkHaRXdqhCNGet61YTXyMvWnfOFfSWr3Mgr8
[Macaron] 2017-07-03 20:05:56: Started GET /api/internal/branch/17/branch for 127.0.0.1
[Macaron] 2017-07-03 20:05:56: Completed /api/internal/branch/17/branch 200 OK in 2.354313ms
[Macaron] 2017-07-03 20:05:56: Started POST /api/internal/push/update for 127.0.0.1
[Macaron] 2017-07-03 20:05:56: Completed /api/internal/push/update 202 Accepted in 61.8573ms
[Macaron] 2017-07-03 20:05:56: Started POST /api/internal/ssh/1/update for 127.0.0.1
[Macaron] 2017-07-03 20:05:56: Completed /api/internal/ssh/1/update 200 OK in 13.464549ms
Jul  3 20:05:56 sshd[27634]: Received disconnect from x.x.x.x port 58096:11: disconnected by user
Jul  3 20:05:56 sshd[27634]: Disconnected from x.x.x.x port 58096

Events from the UI

Now the log when deleting the branch (git push origin :branch)

Jul  3 20:05:59 sshd[27690]: Accepted publickey for git from x.x.x.x port 58098 ssh2: RSA SHA256:BTv+kWSXkHaRXdqhCNGet61YTXyMvWnfOFfSWr3Mgr8
[Macaron] 2017-07-03 20:05:59: Started GET /api/internal/branch/17/branch for 127.0.0.1
[Macaron] 2017-07-03 20:05:59: Completed /api/internal/branch/17/branch 200 OK in 2.063422ms
[Macaron] 2017-07-03 20:05:59: Started POST /api/internal/push/update for 127.0.0.1
[Macaron] 2017-07-03 20:05:59: Completed /api/internal/push/update 202 Accepted in 9.581596ms
[Macaron] 2017-07-03 20:05:59: Started POST /api/internal/ssh/1/update for 127.0.0.1
[Macaron] 2017-07-03 20:05:59: Completed /api/internal/ssh/1/update 200 OK in 4.247786ms
Jul  3 20:05:59 sshd[27690]: Received disconnect from x.x.x.x port 58098:11: disconnected by user
Jul  3 20:05:59 sshd[27690]: Disconnected from x.x.x.x port 58098

No events in the UI, no events in request bin

@bkcsoft does that help?

[lunny]

I think that's a feature, see the description on the webhook on Gitea. There is no delete-branch.

[stephenc]

Says "git push to a repository" so should be all git pushes, even those that remove refs.

That's a bug if you ask me.

A feature would be a separate new event for "delete" but the "push" should include removal based on its description

[stephenc]

@lunny

Just says Git push to a repository it doesn't say Subset of git pushes to a repository that add or update things but not those that remove things

This is clearly a bug

[stephenc]

If you want to have a new event for non-push that is a delete event, that would be a separate feature request, but this issue is PUSH event hooks not sent for branch removal

[bkcsoft]

I would separate these checkboxes into categories, so you get somethings like this. WDYT?

• Git
  • Push event (new commits)
  • Branch events (create, delete)
  • Tag events (create, delete)
• PullRequest
  • Opened
  • Closed
  • [...]
• Issues
  • Opened
  • [...]

[stephenc]

I think push should align with the GitHub push event, i.e. Branch/tag CRUD

The other stuff would be a great feature request, but missing the D on this is a bug