Skip to content

Remove ZAP target ping

Avielle Wolfe requested to merge remove-zap-target-ping into master

What does this MR do?

Removes DAST's second target access check, which used ZAP to ping the target.

  • Removes ZAProxy.ping_target
  • Removes TargetNotAccessibleError
  • Updates E2E tests because the ZAP target ping no longer adds the target to the attack surface

What are the relevant issue numbers?

gitlab-org/gitlab#257963 (closed)

Does this MR meet the acceptance criteria?

Edited by Avielle Wolfe

Merge request reports