Issue Details (XML | Word | Printable)

Key: WEBSOCKET_SPEC-166
Type: Bug Bug
Status: Resolved Resolved
Resolution: Fixed
Priority: Major Major
Assignee: dannycoward
Reporter: markt_asf
Votes: 0
Watchers: 0
Operations

If you were logged in you would be able to see more operations.
websocket-spec

New method Session#getLocal()

Created: 08/Mar/13 11:03 AM   Updated: 19/Mar/13 10:11 PM   Resolved: 19/Mar/13 10:11 PM
Component/s: None
Affects Version/s: None
Fix Version/s: None

Time Tracking:
Not Specified

Tags: final
Participants: dannycoward and markt_asf


 Description  « Hide

Adding the following method to the session would help with error handling as it would provide access to the onError method of the Endpoint:
public Endpoint getLocal()

I'd find this particularly useful when handling DecodeExceptions



dannycoward added a comment - 11/Mar/13 11:01 PM

Hey Mark: Where would you need to call this method from ?

From the developers point of view, all access to the Session occurs within the (local) endpoint the underlying connection is connected to. So, you already know the local endpoint: its this.

I can see that inside the implementation this would be a useful call to have. But, if it is only called by the implementation, why can it not be an implementation specific method on the implementation of Session ?

  • d

markt_asf added a comment - 12/Mar/13 02:04 PM

I use it in a custom message handler to signal a (potentially) non-fatal error to the endpoint. I have no other way to navigate from MessageHandler to Endpoint.

It is a nice to have rather than a must have.


dannycoward added a comment - 19/Mar/13 10:11 PM

OK - thanks. Then I think we will not add this, per our exchange on the expert group.