search

ashenchowthee / zaproxy

Automatically exported from code.google.com/p/zaproxy
0 stars 0 forks source link

Code review request #1561

Closed GoogleCodeExporter closed 9 years ago

GoogleCodeExporter commented 9 years ago 
Purpose of code changes on this branch:
Issue 1559 : Fixed mandatory fields for setScannerAlertThreshold for 
setPolicyAlertThreshold

When reviewing my code changes, please focus on:
Constructor in ActiveScanAPI class for the changes made to mandatory parameters.

        this.addApiAction(new ApiAction(ACTION_SET_POLICY_ALERT_THRESHOLD,
                new String[] { PARAM_ID, PARAM_ALERT_THRESHOLD }, new String[] {PARAM_SCAN_POLICY_NAME}));

        this.addApiAction(new ApiAction(ACTION_SET_SCANNER_ALERT_THRESHOLD,
                new String[] { PARAM_ID, PARAM_ALERT_THRESHOLD }, new String[] {PARAM_SCAN_POLICY_NAME}));

After the review, I'll merge this branch into:
/trunk
/2.4

Original issue reported on code.google.com by dmet...@gmail.com on 12 Mar 2015 at 12:52

Attachments:

GoogleCodeExporter commented 9 years ago 
That looks good :)

Original comment by psii...@gmail.com on 12 Mar 2015 at 9:57