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

DAST #56

Open
wants to merge 4 commits into
base: develop
Choose a base branch
from
Open

DAST #56

wants to merge 4 commits into from

Conversation

yashviagrawal
Copy link
Contributor

Title Value
Type Feature / Bugfix / Refactor / Documentation
Ticket/Issue NA / <Jira ticket or Github issue link>
Migrations No / <apps introducing migrations (eg: core, organization)>
Migration Scripts No / Yes (provide script in description)
ENV vars change No / <list of env vars added or removed (eg: APPKNOX_SAMPLE_ENV=local_dev_value)>
Frontend No / <irene PR link>
Local testing Pending / Done
Staging testing Pending / Skip / <preview link>
On premise notes NA / <any notes specific to on premise deployments>
Documentation None / <Confluence design specification document link>
Release notes None / <Confluence changelog document link>
Version upgrade Major / Minor / Patch

Changelog

  1. Add something
  2. Modify something
    sample code
  3. Remove something
    • item 1
    • item 2

Dependent PRs

cmd/dast_automation.go Outdated Show resolved Hide resolved
cmd/dast_automation.go Outdated Show resolved Hide resolved
helper/clientInitialize.go Outdated Show resolved Hide resolved
helper/dynamic.go Outdated Show resolved Hide resolved
helper/dynamic_check.go Outdated Show resolved Hide resolved
helper/dynamic_check.go Outdated Show resolved Hide resolved
appknox/dynamic_scans.go Outdated Show resolved Hide resolved
cmd/dastcheck.go Outdated Show resolved Hide resolved
helper/dynamic_check.go Outdated Show resolved Hide resolved
helper/dynamic_check.go Outdated Show resolved Hide resolved
Combined logic for schedule automation
Integrated scan status command
Added testing
Final checks made
Final reviewed changes done
Copy link
Contributor

@rajan262 rajan262 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Looks good

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

Successfully merging this pull request may close these issues.

2 participants