MUC support

[genofire]

I found my old code again ... yaja- most of the handling is a little bit trash.

But the library of structs is still not bad (here: xmpp)

The parsing is really easy: https://dev.sum7.eu/genofire/yaja/blob/master/client/comm.go#L9-37.

Another nice feature is, that it is not necessary to iterate over all Message.Extensions we could just make a if PresenceClient.MUC !=nil because omitempty pointer are filled with nil if the values for this structs are not given.

atm i miss somethings to replace other libraries in related project:

• muc support
• obb (to publish image urls as http_upload)
• html body

[mremond]

Thanks for the pointer, I will have a look.
Let's try to make it possible for you to use the same underlying client for all your code base :)

[genofire]

FYI there are:

• hook2xmpp based on mattn
• logmania - also really hacky based on yaja
• thrempp - based on this library ;)

[mremond]

The main advantage of extensions is that they are extensible by the developer. You can use the lib and use your custom namespace extension.

We could have a getter for extensions, a bit similar to your nil test, that could look like (peudo-code):

	var muc MUC
	if ok := p.Get(&MUC); ok {
		// Handle MUC
	}

Anyway, I am still torn between simplicity of your approach and trying to build something that is a bit more complex but could handle more developers' needs ...

[genofire]

Sounds nice - i really do not like this code: code snipped from thrempp

[mremond]

Ok thanks for feedback, I will try to prepare something as discussed during the week-end :)

[mremond]

Here is what the code looks like for Message extension: 4a4c485

I will now expand Presence to have []PresExtension and implement a similar Get method.

[mremond]

I now think that oob and MUC presence extension should be ok for your needs.
I created #68 to track the development of XHTML extension support on messages.

[mremond]

@genofire I have added basic HTML support (see #69). You should have everything to port your code hopefully.
Please, open new issue if you think we are still missing important features.

[genofire]

#69 is for MUC not HTML

[mremond]

Oups, sorry, it is #70

[genofire]

Supi - now i need time to migrate by Programms ;)

Thank you for developing and maintaining xmpp library, it is really useful

[mremond]

Thanks for your feedback and your help :)

And no rush / no pressure to upgrade your programs, I have plenty to do as well.