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.

Sign up for GitHub

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

Jump to bottom

Add SAST advance scan #919

Merged
merged 35 commits into from
Sep 3, 2023
Merged

Add SAST advance scan #919

merged 35 commits into from
Sep 3, 2023

Conversation

attiasas
Copy link
Contributor

@attiasas attiasas commented Aug 28, 2023 •
edited
Loading

  • All tests passed. If this feature is not already covered by the tests, I added new tests.
  • All static analysis checks passed.
  • This pull request is on the dev branch.
  • I used gofmt for formatting the code before submitting the pull request.

Add SAST advance scan:
image

omerzi and others added 17 commits August 10, 2023 17:50
@omerzi
Refactor advanced security scanners
bee1ad6
@omerzi
Merge branch 'dev' into refactor-scanners
f437253
@omerzi
Refactor advanced security scanners
99dd032
@omerzi
Merge remote-tracking branch 'origin/refactor-scanners' into refactor…
ca7dc50 
…-scanners
@omerzi
Merge remote-tracking branch 'upstream/dev' into refactor-scanners
b60e877 
# Conflicts:
#	xray/audit/jas/applicabilitymanager.go
#	xray/audit/jas/applicabilitymanager_test.go
#	xray/audit/jas/iacscanner.go
#	xray/audit/jas/iacscanner_test.go
#	xray/audit/jas/jasmanager.go
#	xray/audit/jas/jasmanager_test.go
#	xray/audit/jas/secretsscanner.go
#	xray/audit/jas/secretsscanner_test.go
#	xray/commands/audit/generic/auditmanager.go
#	xray/utils/analyzermanager.go
@omerzi
Fix tests
97adf0b
@omerzi
Merge remote-tracking branch 'upstream/dev' into refactor-scanners
f477ff5 
# Conflicts:
#	xray/audit/jas/iacscanner.go
#	xray/audit/jas/iacscanner_test.go
#	xray/audit/jas/secretsscanner.go
#	xray/audit/jas/secretsscanner_test.go
@omerzi
update core
1e9e5fd
@omerzi
update core
b666f49
@omerzi
add init func to download analyzer manager
4042ff0
@omerzi
add init func to download analyzer manager
b44f607
@omerzi
add init func to download analyzer manager
92c0de4
@attiasas
initialize implementation of zeroday
710580a
@attiasas
Merge remote-tracking branch 'upstream/dev' into add_zeroday_scan
67cd592
@attiasas
fix merge
854f55a
@attiasas
fix merge
8f6bd75
@attiasas
Add SAST scan
3719410
@attiasas attiasas added the new feature Automatically generated release notes label Aug 28, 2023
@attiasas attiasas changed the base branch from master to dev August 28, 2023 10:35
@attiasas attiasas requested review from sverdlov93 and yahavi August 28, 2023 11:33
@sverdlov93 sverdlov93 requested a review from omerzi August 29, 2023 11:49
@attiasas attiasas requested a review from eyalbe4 August 30, 2023 09:39
sverdlov93
sverdlov93 reviewed Aug 30, 2023
View reviewed changes
xray/audit/jas/applicabilitymanager.go Outdated Show resolved Hide resolved
xray/audit/jas/iacscanner.go Outdated Show resolved Hide resolved
xray/audit/jas/iacscanner.go Outdated Show resolved Hide resolved
xray/audit/jas/iacscanner_test.go Outdated Show resolved Hide resolved
xray/audit/jas/jasmanager.go Show resolved Hide resolved
xray/audit/jas/jasmanager_test.go Outdated Show resolved Hide resolved
xray/audit/jas/sastscanner.go Outdated Show resolved Hide resolved
xray/audit/jas/sastscanner.go Outdated Show resolved Hide resolved
xray/audit/jas/sastscanner_test.go Show resolved Hide resolved
xray/utils/analyzermanager.go Outdated Show resolved Hide resolved
@attiasas attiasas temporarily deployed to frogbot September 3, 2023 07:51 — with GitHub Actions Inactive
@github-actions
Copy link
Contributor

github-actions bot commented Sep 3, 2023

@attiasas attiasas merged commit 54ee983 into jfrog:dev Sep 3, 2023
8 checks passed
@attiasas attiasas added ignore for release Automatically generated release notes and removed new feature Automatically generated release notes labels Sep 3, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@omerzi omerzi omerzi left review comments

@sverdlov93 sverdlov93 sverdlov93 left review comments

@yahavi yahavi Awaiting requested review from yahavi

@eyalbe4 eyalbe4 Awaiting requested review from eyalbe4

Assignees
No one assigned
Labels
ignore for release Automatically generated release notes
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
3 participants
@attiasas @sverdlov93 @omerzi