There is an issue that I am facing while sending Images through XMPP.Below is the code snippet that i have done. I don\'t know what is wrong with the code .
Mess
You can use any name and namespace for your custom packets, but XML stream of your outgoing packets should be valid and well-formed XML document. In your example I see unbound ftpurl, httpurl, imageName nodes. Any good XML/XMPP parser will throw error, because your stream is not well-formed. Common practice is - wrap all you custom nodes in one top-level node and define namespace, like this:
<i xmlns="my:image:transfer">
<httpurl>http://my.image.jpg</httpurl>
<ftpurl>ftp://my/image.jpg</ftpurl>
</i>
This will cause XML parser to treat all your custom XML nodes as they come from "my:image:transfer" namespace and XML stream will be valid.
In the context of your "Image transfer extension" - you are trying to reinvent "Out-of-Band" XMPP File Transfer, which has well-known XMPP Extension - http://xmpp.org/extensions/xep-0066.html
Packets with OOB extension look like that:
<message from='stpeter@jabber.org/work'
to='MaineBoy@jabber.org/home'>
<body>Yeah, but do you have a license to Jabber?</body>
<x xmlns='jabber:x:oob'>
<url>http://www.jabber.org/images/psa-license.jpg</url>
<desc>Jabber license</desc>
</x>
</message>
Smack PacketExtension for this type of payload should look like:
public class OutOfBandData implements PacketExtension {
String description;
String url;
@Override
public String getElementName() {
return "x";
}
@Override
public String getNamespace() {
return "jabber:x:oob";
}
@Override
public String toXML() {
StringBuilder builder = new StringBuilder();
builder.("<" + getElementName() + " xmlns=\"" + getNamespace() + "\">");
if (url != null) {
builder.append("<url>").append(url).append("</url>");
}
if (description != null) {
builder.append("<desc>").append(description).append("</desc>");
}
builder.append("</" + getElementName() + ">");
return builder.toString();
}
public String getDescription() {
return description;
}
public void setDescription(String description) {
this.description = description;
}
public String getUrl() {
return url;
}
public void setUrl(String imageUrl) {
this.url = imageUrl;
}
This is not far from your implementation, but chances where other XMPP clients understand your "Image Transfer" are growing.