Merge branch 'propagation' into 'master'

lumi created 8 years ago

Fix some propagation issues

See merge request !15

Change summary

src/plugins/ping.rs   | 3 ++-
src/plugins/stanza.rs | 3 +++
2 files changed, 5 insertions(+), 1 deletion(-)

Detailed changes

src/plugins/ping.rs

@@ -49,9 +49,10 @@ impl PingPlugin {
                     from: iq.from.unwrap(),
                     id: iq.id.unwrap(),
                 });
+                return Propagation::Stop;
             }
         }
-        Propagation::Stop
+        Propagation::Continue
     }
 
     fn reply_ping(&self, ping: &PingEvent) -> Propagation {

src/plugins/stanza.rs

@@ -38,6 +38,9 @@ impl StanzaPlugin {
         } else if elem.is("iq", ns::CLIENT) {
             let iq = Iq::try_from(elem).unwrap();
             self.proxy.dispatch(iq);
+        } else {
+            // TODO: handle nonzas too.
+            return Propagation::Continue;
         }
 
         Propagation::Stop