kasunkv / owasp-zap-vsts-task

Visual Studio Team Services build/release task for running OWASP ZAP automated security tests

MIT License

30 stars 11 forks source link

fix "Cannot read property 'riskcode' of undefined" #28

Closed insideou7 closed 5 years ago

insideou7 commented 5 years ago

Fixes the following:

27

25

21

16

12

andydkelly commented 5 years ago

Any ideas when we can see this pull request merged?

izvictor commented 5 years ago

Hello, We are currently having the same issue, when are you planing to merge the request ?

kasunkv commented 5 years ago

@insideou7 @izvictor I will work on getting this PRs merged and a new version released by the end of the weekend. I need to do some changes to the CI pipeline and how I release these new versions. I will work on that during the weekend. Sorry for the delay.

andydkelly-ig commented 5 years ago

Thanks @kasunkv appreciate you getting back to us. Looking forward to the fix 👍

jodachecallan commented 5 years ago

@kasunkv Any progress on getting this PR complete?

kasunkv commented 5 years ago

@insideou7 Thanks for the contribution :) Finally got some time to setup the CI process for the repo. Can you please update this branch with the current changes to the master branch and update the PR. Then I will approve and get a version released. Sorry for the delay in response.

insideou7 commented 5 years ago

Done. One problem though- my change is for what the JS needs to be, I don't know what the TS needs to be to compile to it. This is my first time using TS.