Details

    • Type: Bug Bug
    • Status: Resolved
    • Priority: Major Major
    • Resolution: Fixed
    • Affects Version/s: None
    • Fix Version/s: None
    • Labels:
      None

      Description

      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

        Activity

        Hide
        dannycoward added a comment -

        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
        Show
        dannycoward added a comment - 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
        Hide
        markt_asf added a comment -

        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.

        Show
        markt_asf added a comment - 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.
        Hide
        dannycoward added a comment -

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

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

          People

          • Assignee:
            dannycoward
            Reporter:
            markt_asf
          • Votes:
            0 Vote for this issue
            Watchers:
            0 Start watching this issue

            Dates

            • Created:
              Updated:
              Resolved: