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

Updated forms-flow-bpm-cd.yml: Added Dynamic url Handling #2537

Closed
wants to merge 17 commits into from

Conversation

sethu-aot
Copy link
Contributor

Integrate URL availability check and dynamic target selection for ZAP scan

  • Added a step to check the availability of DEV and QA URLs from secrets.
  • Based on the availability, dynamically set the target environment (dev or qa).

@sethu-aot sethu-aot requested review from a team as code owners January 21, 2025 06:49
token: ${{ secrets.GITHUB_TOKEN }}
docker_name: 'ghcr.io/zaproxy/zaproxy:stable'
target: ${{ secrets.BPM_TARGET_URL }}
target: ${{ env.target }}
Copy link
Contributor

Choose a reason for hiding this comment

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

is this a newly added env variable ? if yes, can we give an appropriate name instead of 'target' ? @sethu-aot
cc @kiran-aot

Copy link
Contributor

Choose a reason for hiding this comment

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

Will check for any existing target url varibale names in secrets or env and update it

@sethu-aot sethu-aot closed this by deleting the head repository Jan 21, 2025
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.

3 participants