[Bug 57788] NPE on session.addMessageHandler

2015-04-13 Thread bugzilla 
https://bz.apache.org/bugzilla/show_bug.cgi?id=57788

Remy Maucherat r...@apache.org changed:

   What|Removed |Added

 Resolution|--- |FIXED
 Status|NEW |RESOLVED

--- Comment #2 from Remy Maucherat r...@apache.org ---
r1672628, will be included in 8.0.22. Of course, it will only display a
different error. Source for better test ?

-- 
You are receiving this mail because:
You are the assignee for the bug.

-
To unsubscribe, e-mail: dev-unsubscr...@tomcat.apache.org
For additional commands, e-mail: dev-h...@tomcat.apache.org

[Bug 57788] NPE on session.addMessageHandler

2015-04-03 Thread bugzilla 
https://bz.apache.org/bugzilla/show_bug.cgi?id=57788

Konstantin Kolinko knst.koli...@gmail.com changed:

   What|Removed |Added

 OS||All

--- Comment #1 from Konstantin Kolinko knst.koli...@gmail.com ---
What are you trying to do here and what do you expect? Is it a real use case? 

(Source code =?)

My expectations are that such call is expected to fail.

For reference, WsSession.addMessageHandler(WsSession.java:198) method called
here is

[[[
@Override
public void addMessageHandler(MessageHandler listener) {
]]]

declared in javax.websocket.Session.

Use of that method is discouraged in favor of WebSocket 1.1 methods

Session.addMessageHandler(ClassT clazz, MessageHandler.PartialT handler)
Session.addMessageHandler(ClassT clazz, MessageHandler.WholeT handler)

-- 
You are receiving this mail because:
You are the assignee for the bug.

-
To unsubscribe, e-mail: dev-unsubscr...@tomcat.apache.org
For additional commands, e-mail: dev-h...@tomcat.apache.org