#1761 lost IndicationURL for IndcationListener.indicationOccured

[chrisreich]

Testing CIM Indications with CIMClient 2.1.2 I run into an issue with the IndicationURL that CIMClient passes when calling IndcationListener.indicationOccured().
The IndicationURL was empty for an indication received from a Brocade CIMOM, but when I debugged into the CIMClient I recognized that in
CIMIndicationHandler.dispatchIndications()
the line
String id = pReader.getMethod().getFile();
returns the complete URL (e.g. "http://clientip:5990/mysubid") but the processing afterwards skips the complete String.
Since the javadoc specifies the complete URL-String to be passed, this should be passed to the Indication client.

[Dave Blaschke]

Ravi working on another project until end of year

[Dave Blaschke]

Discussed possible solutions with Christoph, he is trying out the preferred patch in TPC environment

[Dave Blaschke]

Seems like the only URL being passed through were ones that started with / but NOT with /cimom - this code dates back to 1.2.0 timeframe. Patch will still strip the / off any URL that starts with / but not /cimom, but will now pass all other URLs through in their entirety.

[Dave Blaschke]

Patch sent for community review. During a 2 week period any exploiter may comment on the patch, request changes or turn it down completely (with good reason). For the time being the patch is part of the "Experimental" branch in CVS.

[Dave Blaschke]

Patch against Exp

[Dave Blaschke]

The community review is completed and we received no substantial critisism. Therefore the patch has been approved and merged into the "HEAD" branch. The next release will pick it up.

[Dave Blaschke]

The patch was picked up by release 2.1.3 and will be closed.