Hi,

We needed to use the latest trunk chat-portlet sooner than later, and as part of that, we had to make some modifications for it to work for us.

Below they are described and included in a patch file, and hopefully these changes can be included/supported in the official 6.1 release whenever it comes out.

Please note the patch file also includes some service generation code which is not different than the trunk. The primary changes are all in the jabber.* package.

In general our use case is that we needed to use it with Google Apps corporate accounts. Our users have corresponding accounts in our google apps corporate account and there are more than one @domainname (sub-domains) under our google apps account so assumptions currently in the trunk about all user's screen names existing within one domain, or screen name reliance only, does not work with Smack -> Google Talk for a corporate account.

Overall we are asking that configurable support for the changes we made be rolled into the planned mainline release of the chat-portlet so that we do not have to maintain our own branch of it.

-------------------------------------------------------------
OVERVIEW OF CHANGES WE MADE
-------------------------------------------------------------

(#1) Changed JabberImpl's sendMessage() to set the TO (destination user's) jabberId = the user's email address rather than just the screen name.

Can an config option be added for this behavior?

(i.e.) String jabberId = toUser.getEmailAddress();



(#2) Altered JabblerImpl.connect()'s Smack ConnectionConfiguration to set the service name equal to the user's email domain (right side of @) which is what is needed to work with google apps accounts.

Can an config option be added for this kind of behavior?

ConnectionConfiguration config = getConnectionConfiguration();
String serviceName = StringUtil.extractLast(user.getEmailAddress(), StringPool.AT);
config.setServiceName(serviceName);
connection = new XMPPConnection(config);



(#3) Altered the JabberImp.connect()'s smack connection login call to pass email address instead of screen name

Can an config option be added for this kind of behavior? Likely could read from the same config for the above.

connection.login(user.getEmailAddress(), password, PortletPropsValues.JABBER_RESOURCE);



(#4) Adjusted JabberMessageListener.processMessage() to retrieve the sender user by email address rather than screen name due to screen names being different (potentially) than the email for the user (which must match the jabberid MINUS the /resource). This change also added a new method on the Jabber interface: public String getEmailAddress(String jabberIdWithResource); plus on the implementing classes. Again the use of looking up by email might need to be controlled by a config property.



(#5) Adjusted the JabberImpl's getStatuses() method to fetch by email instead of screen name (same reasons as above) as well as catch Exceptions and do a null check in calls to "UserLocalServiceUtil.getUserByEmailAddress()" when a user returned by the XMPP server does not exist in the local liferay install. It seems legitimate that someone might connect this to an XMPP server where users in someone's roster are not in the local Liferay install. This should fail without error, (log/warn maybe) but not cause it to bomb out. Just filter the user from the roster returned. (i.e. _log.warn("XMPP Roster user: " + rosterEntry.getUser() + " not in local Liferay... skipping adding to chat roster..");)



(#6) We front our liferay install via an SSO mechanism and have a custom AuthLogin hook implemented. Subsequently the LoginPostAction in com.liferay.chat.hook.events.LoginPostAction has no password available to us for Smack to consume in its login calls. We made a modification in our copy to fetch it via other means as Smack requires it. However it would be nice if the official chat-portlet release had access to the password OR provided another hook that would permit us to get it via another means available in our AuthLogin hook. Is the login.events.post configuration configurable outside of the portlet WAR? If so this may suffice.



(#7) We backported this to run in 6.0 EE SP1. To do that we had to modify various timer/delay related JS calls in main.js to use AUI DelayedTask instead of the newer debounce methods.


(#8) To work with google talk our portlet.properties ended up looking like this

jabber.enabled=true
jabber.import.user.enabled=false
jabber.host=talk.l.google.com
jabber.port=5222
jabber.resource=
jabber.service.name=
jabber.sock5.proxy.enabled=false
jabber.sock5.proxy.port=-1