Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Support for TomEE #396

Open
padcom opened this issue Sep 8, 2014 · 2 comments
Open

Support for TomEE #396

padcom opened this issue Sep 8, 2014 · 2 comments

Comments

@padcom
Copy link
Member

padcom commented Sep 8, 2014

From [email protected] on January 13, 2014 22:57:01

TomEE is overriding server info so https://code.google.com/p/psi-probe/source/browse/trunk/tomcat70adaptor/src/main/java/com/googlecode/psiprobe/Tomcat70ContainerAdaptor.java test fails. Here is what tomee provides by default: "Apache Tomcat (TomEE)/7.0.50 (1.6.1-SNAPSHOT)".

A workaround is to add the system property tomee.keep-server-info=true but then you loose the previous feature (useful to know what is a tomcat or a tomee in a cluster/infra)

Original issue: http://code.google.com/p/psi-probe/issues/detail?id=396

@padcom
Copy link
Member Author

padcom commented Sep 8, 2014

From [email protected] on January 14, 2014 08:22:43

Issue 397 has been merged into this issue.

@padcom
Copy link
Member Author

padcom commented Sep 8, 2014

From [email protected] on January 28, 2014 07:23:23

There is another workaround in place for this too: https://code.google.com/p/psi-probe/wiki/Troubleshooting#Error_on_first_request However, I think we can support TomEE. Maybe I'll take the opportunity to move those strings outside the adapter and into the Spring configuration.

Summary: Support for TomEE (was: support tomee out of the box)
Status: Accepted
Labels: -Priority-High -Milestone-TBD Priority-Medium Milestone-2.4.0

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

1 participant