zap-extensions icon indicating copy to clipboard operation
zap-extensions copied to clipboard

Published 20 hours ago • verified publisher icon zaproxy

Fix zaproxy/zaproxy #1405 - Field Enumeration

Open AnamikaD opened this issue 7 years ago • 11 comments

Addon for Field Enumeration

Ref: zaproxy/zaproxy#1405.

AnamikaD avatar Jul 11 '17 17:07 AnamikaD

@kingthorin note that this was not yet intended for review, it was just to keep us updated with the latest changes.

thc202 avatar Jul 11 '17 19:07 thc202

Oh, sorry. Jumped the gun ;)

kingthorin avatar Jul 11 '17 20:07 kingthorin

@kingthorin @thc202 @psiinon Please review this!

AnamikaD avatar Sep 04 '17 16:09 AnamikaD

Doing that, will add comments shortly.

thc202 avatar Sep 04 '17 17:09 thc202

@thc202 @kingthorin Thank you for the review. Except for "abstract frames" and code modification in terms of logic, all are done. I have clicked on thumbs up icon to indicate completed tasks

AnamikaD avatar Sep 05 '17 18:09 AnamikaD

@thc202 @kingthorin @psiinon Please review the final changes!

AnamikaD avatar Sep 12 '17 18:09 AnamikaD

As discussed via IRC this could also benefit from a thread pool.

kingthorin avatar Mar 13 '18 17:03 kingthorin

The pull request is now updated (after zap-extensions build changes, zaproxy/zaproxy#5302), also, changed to target ZAP 2.7.0, removed the Commons CSV library (provided by ZAP) and normalised the license and style/format of the code.

thc202 avatar Apr 08 '19 14:04 thc202

Rebased current.

kingthorin avatar Jun 15 '20 18:06 kingthorin

This pull request introduces 2 alerts when merging 67d10150e515b91cacb477e30d0342aea9219c07 into a5d3903b0b36a5980ef87ddaa84ebf2593902b8c - view on LGTM.com

new alerts:

  • 1 for Self assignment
  • 1 for Comparison of identical values

lgtm-com[bot] avatar Jun 16 '20 18:06 lgtm-com[bot]

Squashed, logging tweaked, a bunch of SAST (Sonar) items addressed, rebased current.

kingthorin avatar Sep 24 '21 01:09 kingthorin