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

Add WebLogic pre-12.x Support #277

Closed
dhoffer opened this issue Apr 11, 2012 · 1 comment
Closed

Add WebLogic pre-12.x Support #277

dhoffer opened this issue Apr 11, 2012 · 1 comment

Comments

@dhoffer
Copy link

dhoffer commented Apr 11, 2012

Email Thread:

I'm deprecating WebLogic native support. If you want to use Atmosphere with WebLogic -- when Servlet 3.0 is not available --, you will now need to install the
org.atmosphere.weblogic.AtmosphereWebLogicServlet

instead of the AtmosphereServlet in web.xml, and add the following dependencies

atmosphere-weblogic.jar

the atmosphere-compat-weblogic.jar is no longer required.

I'm doing that because I want to get rid of WebLogic compat dependencies and also make the code more readable.

this only apply to WebLogic and native Comet Support. If you are using WebLogic 12 and up (the one that supports Servlet 3.0), you don't need that. Anyway I don't think a lot of people are using WebLogic ... if you are using a version lower than 12, Atmosphere will still works but will use BlockingIOCometSupport.

I just checked...we have users currently using WebLogic 10.3.2...they might upgrade and might not. I'd like Atmosphere to work in optimal mode with that version too. It's best if this is done via the normal maven dependencies else it has to be documented well how to manually configure for older versions of WebLogic.

Well I may revisit the decision but clearly the code was being really messy and dependencies complex just for a container that isn't used a lot by the community. Just file an RFE to kept track of the discussion and let's see what can be done after 0.9.

@jfarcand
Copy link
Member

Since there is a way to make it work, I will not re-add the code to 1.0.x, so closing this bug.

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

2 participants