Application Gateway without WAF block.

[harkirat22]

As per the Azure Best Practices. Found that Checkov scan didn't find this misconfiguration

[schosterbarak]

Thanks @harkirat22 . would you like to contribute that piece to checkov?

[harkirat22]

Yes. Always up for contributions.

…

On Tue, 20 Oct 2020 at 9:17 am, Barak Schoster Goihman < ***@***.***> wrote: Thanks @harkirat22 <https://github.com/harkirat22> . would you like to contribute that piece to checkov? — You are receiving this because you were mentioned. Reply to this email directly, view it on GitHub <#46 (comment)>, or unsubscribe <https://github.com/notifications/unsubscribe-auth/AENTBKFDCQXBZ3ANPMIIRNDSLS3HTANCNFSM4SVQVUYQ> .

[schosterbarak]

@harkirat22 very cool.
We've done a short guide on it: https://www.checkov.io/5.Contribution/New-Check.html
Also, you are invited to join http://slack.bridgecrew.io/ and talk with @metahertz or myself if you want to ask a question along the way.

[harkirat22]

Thank you @schosterbarak for the invite. I'll read the guide and do the contribution. Moreover, there are a couple of more checks that I have documented in my notes and they need to be in checkov.

[schosterbarak]

@harkirat22 please do. feel free to reach out :)

[metahertz]

Echoing @schosterbarak, Thanks for raising this and feel free to reach out to either of us with any issues or questions working with the Checkov codebase! Copying existing similar checks and matching tests are a great starting point after https://www.checkov.io/5.Contribution/New-Check.html :)