Thank you very much for your quick response.
On 4 July 2013 00:41, Marek Posolda <mposolda(a)redhat.com> wrote:
PR for both changes (upgrade twitter4j-core and adding Valve for
Tomcat)
is here
https://github.com/gatein/gatein-portal/pull/551 . After merging
it, you shouldn't have issues anymore.
Marek
On 3.7.2013 16:40, Marek Posolda wrote:
Hi Trong,
On 3.7.2013 14:16, Trong Tran wrote:
Hi,
I'm playing OAuth integration feature with GateIn Tomcat packaging.
Unfortunately it doesn't work for me.
I'm testing with latest code of GateIn master. It doesn't login into
portal after registration, the login form with a message "Sign in failed.
Wrong username or password." is displayed instead.
There is one additional needed thing for Tomcat integration and seems that
I forgot to add it to docs :-\ You need to add ServletAccessValve into
conf/context.xml or conf/server.xml (host section) as mentioned in
https://docs.jboss.org/author/display/GTNPORTAL36/Central+Authentication+...
The valve is needed for SSO and OAuth integration. It's used to add
HttpServletRequest to ThreadLocal before JAAS authentication is triggered,
so that Login modules have access to current HttpServletRequest. In JBoss,
it's automatically possible via
javax.security.jacc.PolicyContext.getContext, however in Tomcat it's not
automatically possible, so we need to manually add it via Valve.
I will update the docs. I am thinking about adding the ServletAccessValve
into Tomcat packaging by default, which would mean one less step for SSO
and OAuth integration on Tomcat. Will likely send PR soon.
Moreover, I have also tested on JBoss packaging. It works well for
Facebook and Google BUT it doesn't work with Twitter.
yes, unfortunately we used older version of twitter4j-core library
(2.2.6), which is using Twitter API 1.0. Twitter recently updated to 1.1
and stopped to support this API from July 2013, so it doesn't work anymore.
The fix is to update twitter4j-core to latest version 3.0.3, which is
using Twitter API v1.1. I will send PR later today after performing some
more tests. The fix will be in JPP 6.1, but unfortunately not in already
released GateIn 3.6 :-(
The workaround for GateIn 3.6 is to upgrade the JAR. You can download
twitter4j-core-3.0.3.jar from
http://search.maven.org/remotecontent?filepath=org/twitter4j/twitter4j-co...
replace the old twitter4j-core-2.2.6.jar in GATEIN_TOMCAT_HOME/lib/ (on
Tomcat7) or in GATEIN_JBOSS_HOME/modules/org/twitter4j/main/ (on JBoss7.
You will need to update version in
GATEIN_JBOSS_HOME/modules/org/twitter4j/main/module.xml as well)
Thanks for the feedback!
Marek
Is there anybody having the same problem ? are this bugs OR I missed
something in configuration ?
Thanks
--
*Trong Tran*
*(+84) 983841909 | *trongtt(a)gmail.com
Twitter:
http://twitter.com/trongtt**
_______________________________________________
gatein-dev mailing
listgatein-dev@lists.jboss.orghttps://lists.jboss.org/mailman/listinfo/gatein-dev
_______________________________________________
gatein-dev mailing
listgatein-dev@lists.jboss.orghttps://lists.jboss.org/mailman/listinfo/gatein-dev