saycorey / memcached-session-manager

Automatically exported from code.google.com/p/memcached-session-manager
0 stars 0 forks source link

memcached-session-manager tc7 1.7.0 is not compatible with tomcat catalina 7.0.25 and below #200

Closed GoogleCodeExporter closed 9 years ago

GoogleCodeExporter commented 9 years ago
What steps will reproduce the problem?
1.upgraded required msm jars from version 1.6.5 to 1.7.0
2.Using Tomcat 7.0.23

What is the expected output? What do you see instead?
Ideally session replication should have worked

What version of the product are you using? On what operating system?
I am as of now using version 1.6.5 with linux fedora operating system

Please provide any additional information below.

Observed error:
Caused by: java.lang.NoClassDefFoundError: 
org/apache/catalina/util/SessionConfig
    at de.javakaffee.web.msm.MemcachedBackupSessionManager.getSessionCookieName(MemcachedBackupSessionManager.java:829)
    at de.javakaffee.web.msm.MemcachedSessionService.startInternal(MemcachedSessionService.java:416)
    at de.javakaffee.web.msm.MemcachedBackupSessionManager.startInternal(MemcachedBackupSessionManager.java:524)
    at org.apache.catalina.util.LifecycleBase.start(LifecycleBase.java:148)
    ... 31 more

Reason found:  

org/apache/catalina/util/SessionConfig was introduced in tomcat-catalina 7.0.26

Original issue reported on code.google.com by VISH.THE...@gmail.com on 4 Mar 2014 at 2:00

GoogleCodeExporter commented 9 years ago
I suggest to upgrade to the latest tomcat 7 release.

Original comment by martin.grotzke on 4 Mar 2014 at 10:38

GoogleCodeExporter commented 9 years ago
Martin, the problem is, its too huge a change, i know your suggestion is valid, 
but the change might have high impact on our systems. Still, will try to 
achieve that. 

Just a suggestion, please add this somewhere in your docs, so that people may 
know, before trying to upgrade.

Thanks,
Vishal Meghani

Original comment by VISH.THE...@gmail.com on 5 Mar 2014 at 4:49

GoogleCodeExporter commented 9 years ago
Hmm, do you have a suggestion where to add this hint?

Original comment by martin.grotzke on 6 Mar 2014 at 12:41

GoogleCodeExporter commented 9 years ago
Btw, you could also submit a pull request that adds some reflection to support 
also older versions of tomcat7.

Original comment by martin.grotzke on 6 Mar 2014 at 12:43

GoogleCodeExporter commented 9 years ago
Martin, i think setupAndConfiguration's Configure tomcat should be the right 
place to add this.

Its a nyc idea, to submit a pull request and use reflection, will do that if 
necessary. :)

Thanks for all your help.

Original comment by VISH.THE...@gmail.com on 6 Mar 2014 at 2:11

GoogleCodeExporter commented 9 years ago
Docs are updated.

Original comment by martin.grotzke on 27 Mar 2014 at 7:56