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

Optimized impact paths building #441

Merged
merged 6 commits into from
Nov 30, 2023

Conversation

asafgabai
Copy link
Contributor

@asafgabai asafgabai commented Nov 23, 2023

  • All tests passed. If this feature is not already covered by the tests, I added new tests.

Depends on jfrog/jfrog-ide-webview#49

Separated the process of grouping dependencies in file nodes from the process of building impact paths.
Improved both processes for better performance.
Impact paths are built bottom-up (from the vulnerable dependency to the root of the project) to avoid scanning the whole dependency tree.
Dependencies of Maven projects are grouped in file nodes in a similar, more efficient way, to find out which modules depend on vulnerable dependencies, without the need to find all possible paths between these dependencies and the roots of all modules.

… process of building impact paths. Improved both processes for better performance.
@asafgabai asafgabai added the improvement Automatically generated release notes label Nov 23, 2023
@asafgabai asafgabai added the safe to test Approve running integration tests on a pull request label Nov 23, 2023
@github-actions github-actions bot removed the safe to test Approve running integration tests on a pull request label Nov 23, 2023
@asafgabai asafgabai added the safe to test Approve running integration tests on a pull request label Nov 23, 2023
@github-actions github-actions bot removed the safe to test Approve running integration tests on a pull request label Nov 23, 2023
Copy link

👍 Frogbot scanned this pull request and found that it did not add vulnerable dependencies.


@asafgabai asafgabai requested a review from Or-Geva November 23, 2023 16:52
@asafgabai asafgabai marked this pull request as ready for review November 23, 2023 16:52
@asafgabai asafgabai requested a review from eyalbe4 November 28, 2023 12:29
@Or-Geva Or-Geva requested a review from yahavi November 29, 2023 08:01
…aph-creation

# Conflicts:
#	src/main/java/com/jfrog/ide/idea/scan/ScannerBase.java
#	src/main/java/com/jfrog/ide/idea/ui/webview/WebviewObjectConverter.java
@asafgabai asafgabai added the safe to test Approve running integration tests on a pull request label Nov 30, 2023
@github-actions github-actions bot removed the safe to test Approve running integration tests on a pull request label Nov 30, 2023
@asafgabai asafgabai merged commit 5a6a4a4 into jfrog:master Nov 30, 2023
12 of 13 checks passed
@asafgabai asafgabai deleted the improve-impact-graph-creation branch November 30, 2023 12:12
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
improvement Automatically generated release notes
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants