/docs.html?path=/capture-admin testing form fails to register Capure Agent

Steps to reproduce

Steps to reproduce:
1. go to /docs.html?path=/capture-admin
2. scroll down to POST /agents/{name}/configuration, and click on "Testing form (click to reveal)"
3. fill in agent's name, and properties:
<<<<<<<<<<<<<<<<<<<<<<<<<<<<<<<<<
<!DOCTYPE properties SYSTEM "http://java.sun.com/dtd/properties.dtd">
<properties/>
<<<<<<<<<<<<<<<<<<<<<<<<<<<<<<

Actual Results:
Status: 400 (Bad Request)

Expected Results:
Status: 200 (OK)

Activity

Show:
Pawel Fic
September 20, 2011, 9:52 AM

Solution is also trivial, just need to change capital 'C' in Configuration field to small 'c' in CaptureAgentStateRestService.java line 268, name = "Configuration". [learning how to add comments ],
Code revision is 10980.

Rubén Pérez
September 20, 2011, 10:21 AM

You're normally suppossed to attach a patch file (made with diff), so that a committer don't have to do the changes by hand.
And we normally write only the version that "fixes" the bug. It's supposed that the code version corresponds to the latest revision in the code version indicated by "Affects versions".

This fix is committed as rev. 10981

Pawel Fic
May 30, 2012, 9:58 AM

Hi,
I have checked this, and it still occurs in 1.4, and there is still capital "C".
I know it's minor bug, but still....
I have patch for 12275, I'll try to attach it.

Rubén Pérez
May 30, 2012, 11:10 AM

Reassigning to Greg, since the patch is easy enough and this is a regression because I had already committed this once.

Greg Logan
May 30, 2012, 6:02 PM

Good catch Pawel, thanks for the patch. I've applied it to trunk with rev 12279, and merged that into 1.3.x with rev 23380 and 1.4.x with rev 12281.

Assignee

Greg Logan

Reporter

Pawel Fic

Severity

Incorrectly Functioning Without Workaround

Tags (folksonomy)

None

Components

Fix versions

Affects versions

Priority

Trivial
Configure