ICTU / zap2docker-auth-weekly

Zap baseline scanner in Docker with authentication
Apache License 2.0
104 stars 70 forks source link

Update zap_auth.py #70

Closed Gomathy-Gopinath closed 3 months ago

erno1978 commented 3 months ago

This PR seems unwanted to me (at least without explanation). In fact it feels quite wrong to replace a variable with a hardcoded secret. As a bonus it's weird that GitGuardian has no problem with this as well. @Gomathy-Gopinath can you explain why you raised this PR or is it by mistake?

erno1978 commented 3 months ago

No comments received. Closing PR.