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

Problem with WeblogicWebSocketHandler #1729

Closed
frezelth opened this issue Sep 26, 2014 · 7 comments
Closed

Problem with WeblogicWebSocketHandler #1729

frezelth opened this issue Sep 26, 2014 · 7 comments

Comments

@frezelth
Copy link

Hi,

I have a problem using atmosphere (websockets) with Weblogic 12.1.2.
In WeblogicWebSocketHandler line 153 in the configure() method there is a call to the config object.
This config object is null at this moment, giving a NullPointerException:

config = config.getBroadcasterFactory().lookup(ROOT_MASTER).getBroadcasterConfig().getAtmosphereConfig();

Regards,

Thomas

@jfarcand
Copy link
Member

Please paste the complete satck trace. Thanks!

@frezelth
Copy link
Author

I'm using it through Vaadin and for any reason I don't see the stacktrace, it might be lost somewhere...

But for sure it fails, because the call to the configure() method is done if config == null in the onOpen() method.

And if config is null, the call to config.getBroadcasterFactory().lookup(ROOT_MASTER).getBroadcasterConfig().getAtmosphereConfig(); fails with a NPE.

I saw that in the first version of the class this line was replaced by: config = BroadcasterFactory.getDefault().lookup("/*").getBroadcasterConfig().getAtmosphereConfig();

Thanks!

@jfarcand
Copy link
Member

Ok Vaadin uses a fork of Atmosphere and install it differently. @jdahlstrom, can you help with this?

@frezelth
Copy link
Author

I already opened a support request to Vaadin for push with weblogic :) But I also investigated myself and thought there was an issue on the atmosphere side with this class.
Thanks

@jfarcand
Copy link
Member

@frezelth Have you tested it outside Vaaadin? For me everything works as expected.

@frezelth
Copy link
Author

I tried the atmosphere chat sample with a brand new weblogic 12.1.2.1 domain.

I downloaded the sources both versions 2.1.1 and latest version 2.2.2. I made a clean install and in both cases I made the changes you mentioned here:
https://github.com/Atmosphere/atmosphere/wiki/WebLogic-12.1.2-WebSocket-Support

What I found is that it works with version 2.1.1 and it does not work with version 2.2.2.

A printscreen of chrome network monitor gives me this:
failed

In the working version I get this:
working

No stracktrace available in the console.

Thanks,

Thomas

@jfarcand jfarcand reopened this Oct 2, 2014
@jfarcand jfarcand added 2.3.0 and removed not a bug labels Oct 2, 2014
@jfarcand
Copy link
Member

jfarcand commented Oct 2, 2014

OK thanks. Fixed in 2.3.x and 2.2.3

jfarcand added a commit that referenced this issue Oct 2, 2014
@jfarcand jfarcand added the 2.2.3 label Oct 2, 2014
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