Re: [gatein-dev] OAuth integration in Tomcat packaging

Hi Trong, On 3.7.2013 14:16, Trong Tran wrote: 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. 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... and 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