Default Project CR-8242

Adding extended url validation feature in WPA4

Closed on 24 Sep 15

  •  
  •  
  •  
  •  
  • Author & Moderator
  • Reviewers
    • Reviewer completed
    • Reviewer completed

CR-8242 33

Keyboard shortcuts  
Summarize the review outcomes (optional)
 
#permalink

Details

Warning: no files are visible, they have all been filtered.
Participant Role Time Spent Comments Latest Comment
Author & Moderator 1h 42m 16 this bit (and all the rest of the following set_valid_url...
Reviewer - Complete 2h 11m 14 okay i understand: current index starts out in the naming...
Reviewer - Complete 27m 3 Just minor comments line. Marking as complete.
Total   4h 19m 33  
#permalink

Objectives

Extended-url validation feature available in agents3 (AME-1132) is ported to the WPA4 pooled executor and timer logic.

#permalink

Issues Raised From Comments

Key Summary State Assignee
#permalink

General Comments

18 Sep 15

Charles Sparey says:

Just minor comments line. Marking as complete.

/source/apache/agent.c Changed  
Open in IDE #permalink
/source/iis/agent.c Changed  
Open in IDE #permalink
/source/varnish/agent.c Changed  
Open in IDE #permalink
/source/varnish3/agent.c Changed  
Open in IDE #permalink
/source/am.h Changed  
Open in IDE #permalink
/source/audit.c Changed   1
Open in IDE #permalink
/source/config.c Changed  
Open in IDE #permalink
/source/config_file.c Changed   2
Open in IDE #permalink
/source/init.c Changed  
Open in IDE #permalink
/source/log.c Changed  
Open in IDE #permalink
/source/naming_valid.c Changed   29
Open in IDE #permalink
/source/utility.h Changed  
Open in IDE #permalink

Review updated: Reload | Ignore | Collapse

You cannot reload the review while writing a comment.

Create Issue

X
Assign To Me

Log time against