search

yikuang / wave-protocol

Automatically exported from code.google.com/p/wave-protocol
0 stars 0 forks source link

WaveXmppComponent.processIqPacket does not handle error responses #42

Closed GoogleCodeExporter closed 9 years ago

GoogleCodeExporter commented 9 years ago 
During disco querying, if the remote server has multiple components and one
or more is inaccessible (returns 404 because it has no DNS entry), the
error response is ignored instead of continuing to the next component.

E.g. remote components: [conference, proxy, pubsub, wave] and only "wave"
has a DNS entry. The processing will stop with "conference" when it returns
404. If this is fixed it then stops at "proxy" instead. Handling
IQ.Type.error as if it were IQ.Type.result works around the problem
(although it is then trying to examine the error response as though it were
a valid result).

Original issue reported on code.google.com by goo...@sa.me.uk on 3 Oct 2009 at 6:37

GoogleCodeExporter commented 9 years ago

Original comment by btkalman@gmail.com on 11 Oct 2009 at 12:00

GoogleCodeExporter commented 9 years ago 
This is fixed in my clone (anthonybaxter-federation) and will be merged into 
the 
mainline in a few hours.

Original comment by anthonybaxter@gmail.com on 30 Oct 2009 at 3:54

GoogleCodeExporter commented 9 years ago

Original comment by anthonybaxter@gmail.com on 5 Nov 2009 at 6:01