-
Notifications
You must be signed in to change notification settings - Fork 28
Add environments list API (AST-129883) #1415
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: main
Are you sure you want to change the base?
Conversation
|
Great job! No new security vulnerabilities introduced in this pull request |
| @@ -0,0 +1,45 @@ | |||
| package commandutils | |||
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.
All these methods are in root.go and mainly used inside command implementations. Do we need this redundant globally available?
| triageGetStatesCmd := triageGetStatesSubCommand(customStatesWrapper, featureFlagsWrapper) | ||
|
|
||
| addFormatFlagToMultipleCommands( | ||
| commandutils.AddFormatFlagToMultipleCommands( |
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.
Can we avoid changes to existing functionality when it can be avoided?
| {RiskManagementPathKey, RiskManagementPathEnv, "api/risk-management/projects/%s/results?scanID=%s"}, | ||
| {ConfigFilePathKey, ConfigFilePathEnv, ""}, | ||
| {RealtimeScannerPathKey, RealtimeScannerPathEnv, "api/realtime-scanner"}, | ||
| {DastEnvironmentsPathKey, DastEnvironmentsPathEnv, "api/dast/scans/environments"}, |
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.
please add new entries at the end and keep them together.

By submitting this pull request, you agree to the terms within the Checkmarx Code of Conduct. Please review the contributing guidelines for guidance on creating high-quality pull requests.
Description
Add an API that allows searching for environments.
This will be used by the new DAST results panel in the extension for displaying DAST results.
Type of Change
Related Issues
https://checkmarx.atlassian.net/browse/AST-129883
Checklist
Screenshots (if applicable)
Add screenshots to help explain your changes.
Additional Notes
Add any other relevant information.