Project

General

Profile

Bug #341

CapsManager implementation has minor issue for the protocol

Added by Haojie Song almost 8 years ago. Updated almost 8 years ago.

Status:
Closed
Priority:
Normal
Category:
-
Target version:
Start date:
01/31/2011
Due date:
% Done:

100%

Estimated time:
Affected version:
hg
Locale:
Close

Description

As per the protocol, the service discovery request should include 'node' attribute (the 'node' value from presence 'c' element plus "#" and 'ver' string). It means should invoke the discoverInfo(String entityID,String node) method of ServiceDiscoveryManager and pass the concatenated 'node' value.

Should modify the validate() method like the below:
private boolean validate(String jid, String ver, String hashMethod, String node) {
try {
DiscoverInfo info = mSdm.discoverInfo(jid, node+"#"+ver);
....
}
}

Associated revisions

Revision 863:e0aa4b3c5d17 (diff)
Added by Frédéric Barthéléry almost 8 years ago

Fix minor issue in CapsManager.
The service discovery request must include the node attribute.
Fix #341

Revision 885:e0aa4b3c5d17 (diff)
Added by Da Risk almost 8 years ago

Fix minor issue in CapsManager.
The service discovery request must include the node attribute.
Fix #341

Revision 864:e0aa4b3c5d17 (diff)
Added by Da Risk almost 8 years ago

Fix minor issue in CapsManager.
The service discovery request must include the node attribute.
Fix #341

History

#1 Updated by Frédéric Barthéléry almost 8 years ago

  • Status changed from New to Assigned
  • Assignee set to Frédéric Barthéléry
  • Target version set to 0.1.6

Thanks for reporting this. It will be fixed soon.

#2 Updated by Frédéric Barthéléry almost 8 years ago

  • Status changed from Assigned to Resolved
  • % Done changed from 0 to 100

Applied in changeset r863.

#3 Updated by Frédéric Barthéléry almost 8 years ago

  • Status changed from Resolved to Closed

Also available in: Atom PDF