Skip to content
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

Support for Dependabot alerts #11

Open
mario-campos opened this issue Jun 14, 2021 · 15 comments
Open

Support for Dependabot alerts #11

mario-campos opened this issue Jun 14, 2021 · 15 comments

Comments

@mario-campos
Copy link

It would be great to include support for the other types of GHAS alerts:

  • Dependabot Security alerts
  • Secret Scanning alerts
@cmboling
Copy link
Contributor

For secret scanning, I'm giving this one another review before approving and merging: #7
For Dependabot, unfortunately we currently delete resolved alerts on our end, so it would be a partial sync, which some people may be fine with, some others not so much. I'd prefer the sync to be consistent as well, so source of truth/historical data should be on the GitHub side. I believe the Dependabot team is working on retaining the resolved alerts which would be helpful for this integration as well as anyone who is looking to view the historical data on resolved alerts.

@cmboling cmboling self-assigned this Aug 16, 2021
@cmboling
Copy link
Contributor

cmboling commented Nov 9, 2021

The secret scanning PR has been thoroughly tested. I'll try to get other feedback this week so it can be merged asap!

@cmboling
Copy link
Contributor

cmboling commented Nov 9, 2021

Resolved Dependabot alerts are still not retained, but there are ways to sync the actual security/updates: https://github.com/namin2/dependabot_jira

@mario-campos
Copy link
Author

@cmboling, why does Dependabot not retaining alerts after being resolved pose a problem?

@cmboling cmboling changed the title Support for Dependabot and Secret Scanning Support for Dependabot alerts Nov 18, 2021
@cmboling
Copy link
Contributor

Howdy @mario-campos!! In the case of Dependabot, I don't see any issues with syncing the Dependabot alerts, but when it comes to having a clear source of truth, it may possibly lead to confusion at first. For example, there would be a Jira issue mapped to some resolved Dependabot alert (because it was open and was synced to Jira at some point) but there's no indication of a resolved alert in the UI or in the endpoint. Because we know how Dependabot security updates treats resolved alerts, we can say/assume the alert was resolved and deleted and implement such logic in this integration. We could definitely make it work, but it's a bit weird to see that inconsistency. For the gh2jira mechanism, the source of truth is on GitHub, so the lack in UI consistency could cause some confusion. If users are doing the bidirectional sync, the Dependabot sync is fine, given the constraints mentioned. But then again, re-opening a Jira issue for a Dependabot alert does not/would not re-open the Dependabot alert in GitHub because the alert doesn't exist and besides, dismissed alerts can't be re-opened anyways. Ok yea my brain hurts 🧠

So if we could retain alerts... THAT WOULD BE THE BEST INTERNET CHRISTMAS GIFT EVER. 🌴
Simplification is key but I am getting many requests to get Dependabot alerts integrated, so I will work on that asap.

For anyone else reading.. apologies for the delay! Hang in there. 💟

But yeah if a user has Dependabot security updates enabled, that's perfect because we can easily sync it, such as the integration mentioned above. Definitely recommend that approach for now if at all possible.

@cmboling
Copy link
Contributor

I also changed the title since we already got the secret scanning stuff merge 😄

@cmboling cmboling moved this from Feature requests to In Progress in GHAS + Issue Tracking Integration Development Nov 18, 2021
@Dylan-Rinker
Copy link

Just checking in the status of this issue/request. I know there has a been changes to dependabot alert persistence and GraphQL API in the recent months, and I was curious if the new features are enough to enable syncing of dependabot alerts now?

@cironera
Copy link

cironera commented Aug 25, 2022

Bump -> Any progress on dependabot alerts?

@felickz
Copy link

felickz commented Aug 30, 2022

Dependabot adding API support in Q3 might remove some blockers here - github/roadmap#495

@Dylan-Rinker
Copy link

With Code Scanning alerts now being part of the Jira Application, does the development effort for Dependabot sync make more sense to occur here or there?

@cfisher281
Copy link

Any update to adding Dependabot alerts?

@codebop
Copy link

codebop commented Mar 22, 2023

Any updates here? This would greatly streamline my Jira workflow.

@codebop
Copy link

codebop commented Apr 14, 2023

We just received a tech win for 300 GHAS licenses (Synk SCA takeout). Dependabot alert integration with JIRA was the one that we failed to deliver on.

@pladuke
What does that mean?

@codebop
Copy link

codebop commented Apr 14, 2023

They ran a comprehensive GHAS evaluation, and the ability to create JIRA tickets for Dependabot alerts was the only unsuccessful piece of the evaluation.

@pladuke Who ran a GHAS evaluation? A customer of yours our a team within GitHub? Does this mean this feature will be worked on?

@CRUZEAAKASH
Copy link

Any progress on dependabot alerts?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Development

No branches or pull requests

8 participants