-
Notifications
You must be signed in to change notification settings - Fork 3.4k
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
Add Support for BitBucket webhooks #5209
Comments
+1 for Bitbucket Server |
+1 for bitbucket |
I try restore this commit with some changes. It fine work with bitbucket 5.12 and AWX 9.1.0 for me. Not tested this with last environment yet. |
Support for this in AWX is blocked, because Bitbucket does not support signed webhooks in its free product. Details at: |
+1 |
Is there any update on this RFE? |
+1 for Bitbucket Server |
@rmauc @jvandermeulen #8328 (comment) as noted above. |
Is waiting for the free product really justification to restrict enterprise customers from fully benefiting from the Tower feature set? |
Why could we not enable bitbucket on-prem (server/datacenter) Webhooks with clear documentation & UI tooltips explaining that this does not work with bitbucket cloud? Having bitbucket Webhooks integration with AWX would massively simplify CI/CD workflows as there's no longer a need for an intermediary service to receive Webhooks and trigger tower-cli commands. I'd be happy to help test anything related to bitbucket on-prem webhook integration. |
Please consider providing this for the on-premise server / datacenter versions. I know a lot of companies that would benefit from this integration. |
I am running into more and more clients that are using Bitbucket and have a pretty clear need for this functionality. Please reconsider supporting it. I would help out with testing as well. thanks. |
+1 for Bitbucket |
+1 for bitbucket |
This is really useful feature. We really need it! However there seems to be some changes and end of life for the old Server edition bitbucket. So you may no longer need to support 2 different bitbucket workflows, which would make adding back this feature a bit easier. |
+1 |
1 similar comment
+1 |
+1 for bitbucket webhooks |
please stop commenting +1. it really just annoys everyone that's watching this issue/repo as they all get an email every time. |
backing out ansible#10524 from release_3.8.4
@ryanpetrello This would sound more reasonable if AWX didn't support GitLab, which also does not support signed webhooks. As of today, |
Hey @philipsd6, I'm less involved with this project than I used to be, and am no longer an active contributor. @shanemcd @AlanCoding @chrismeyersfsu ☝️ |
Good to know! Thanks for all your contributions and support in the past! |
Just an update for folks. We are not ignoring your requests here, but we will not be adding this until Bitbucket handles the underlying issue on their end. I encourage you to weigh in on these related issues: |
@shanemcd but what about Bitbucket Server and Datacenter? Support for Webhook Secrets is already implemented there. And it will probably take some time for atlassian to implement it in the cloud version. 😕 |
We do not have a budget for purchasing licenses to test and support integrations with enterprise software. Maybe if Atlassian wanted to hook us up we could consider it? But I find it morally questionable they chose to withhold a critical security feature from their cloud-based offering. |
Atlassian has (currently) only chosen to support IP allow lists, so at least they are supporting reducing the attack surface. I and others have encouraged Atlassian to support signed webhooks. But @shanemcd, I respectfully disagree with the decision here for BitBucket, for the following reasons:
My proposal is that we/I add a FWIW, the allowlist can be retrieved (using HTTPie and Jello): http GET https://ip-ranges.atlassian.com/ \
| jello -lr '[item["cidr"] for item in _["items"]
if "bitbucket" in item.get("product", "")
and item["direction"] == ["egress"]
and item["mask_len"] < 32]' |
@philipsd6 Well thank you for being respectful about it. 😄 I am not sure how I missed your comment above, I apologize. I just looked in the bitbucket cloud documentation and do not see anything in the payload similar to If you are willing to send a PR that implements your proposal we'll give it an honest review and consideration. We also have someone who has reached out to Atlassian to see if they'd be willing to give us a license for testing. |
No @shanemcd, you definitely didn't miss anything! What I was saying was that one could have the webhook url be It does imply that perhaps there should be two Webhook Services listed: I am willing to submit a PR but my company workload is currently very high. I'll try to get a draft PR going next week. |
@philipsd6 As an Atlassian, I second your suggestion (implication) that there should be 2 options, separating Bitbucket Cloud from Bitbucket Data Center. What may not be obvious from the branding is these are 2 very different products, maintained by 2 different Atlassian teams. There's only informal connection between the features of one to the other. Hence, the shape of webhooks are quite different between the 2 products and it would be worth treating the 2 as if they were different products. It looks like you have a possible path forward for Bitbucket Data Center, which seems the more popular need based on comments here. I wonder if it would be worth renaming this issue for Bitbucket Data Center and opening a new issue for Bitbucket Cloud? For Bitbucket Cloud, there is a JWT-based webhook authentication mechanism when using Bitbucket Connect. The Bitbucket Connect architecture would have considerable overhead for AWX but, if someone does have ideas how it could work, feel free to at-mention me for input. Given the UI-based webhooks would be an easier means of integration, I have commented privately on the issues to prompt action. |
+1 for BitBucket |
Howdy folks, This PR has been open for quite a while, and it's significant functionality. Any chance this could be revisited soon? I hate having to involve a jenkins instance to do what bitbucket server could do directly with AWX. |
To any one impatient to wait for this implementation, I managed to have bitbucket webhooks working by utilising nginx's "proxy_set_header" that sits in front of AWX. It's not an ideal solution but allows you to send the payload to awx job for processing. The gist of it is: Enable GitLab webhook which will give you a API link as well as token for authentication. On nginx add, customised to your set up of course: location ~ [jobid]/gitlab/ { And Bob's your uncle. The payload is sent to AWX as an extra var under tower_webhook_payload. The nginx directives will need to be added for each job that requires webhook capabilities. It's a bit of a hack I guess but been nearly 4 years since this request was made. |
ISSUE TYPE
SUMMARY
Restore commit 885841c to add support for BitBucket webhooks, since Server edition supports signed payloads and Cloud edition suggests IP Whitelists (manage webhooks and ip whitelist)
@jbradberry
The text was updated successfully, but these errors were encountered: