-
Notifications
You must be signed in to change notification settings - Fork 42
feat: sast server #650
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
base: dev
Are you sure you want to change the base?
feat: sast server #650
Conversation
attiasas
left a comment
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Nice job! Take a look at my comments, in addition:
- Please add a bit more description to the PR about this command
- After AM is released with the new cmd, make sure tests are passing
| @@ -0,0 +1,15 @@ | |||
| package sast_server | |||
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Lets make sure technical writer goes over the text and approved by PM
| Enrich = "sbom-enrich" | ||
| UploadCdx = "upload-cdx" | ||
| MaliciousScan = "malicious-scan" | ||
| SastServer = "sast-server" |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
assign flag to command at commandFlags in the file
cli/scancommands.go
Outdated
| return err | ||
| } | ||
|
|
||
| port := c.GetStringFlagValue("port") |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
| port := c.GetStringFlagValue("port") | |
| port := c.GetStringFlagValue(flags.Port) |

devbranch.go vet ./....go fmt ./....Add local sast server command