Skip to content

[MIRRORED] DO NOT MERGE: fix(bitbucketdatacenter): ignore branch deletion events in ParsePayload #2130

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Draft
wants to merge 2 commits into
base: main
Choose a base branch
from

Conversation

chmouel
Copy link
Member

@chmouel chmouel commented Jun 13, 2025

Mirrors #2126 to run E2E tests. Original author: @infernus01

@zakisk zakisk force-pushed the test-pr-2126-infernus01 branch from f9f5a41 to 7f614d2 Compare June 13, 2025 11:02
@chmouel chmouel force-pushed the test-pr-2126-infernus01 branch from 7f614d2 to f9f5a41 Compare June 13, 2025 11:31
@zakisk zakisk force-pushed the test-pr-2126-infernus01 branch 2 times, most recently from b1fd690 to 6618555 Compare June 16, 2025 09:50
When a branch is deleted in Bitbucket Data Center, the repo:refs_changed
webhook sends a change with toHash set to all-zeros. Previously, Pipelines
as Code treated this like a push event, which caused errors when trying
to validate a missing commit.

This patch detects and ignores such events early in ParsePayload.

Fixes openshift-pipelines#2035
@chmouel chmouel force-pushed the test-pr-2126-infernus01 branch from 6618555 to bd8c88e Compare June 19, 2025 14:26
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Development

Successfully merging this pull request may close these issues.

2 participants