Details

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

      Description

      Allow to add listener for major event (such as when |close| method is fired) about SSH Session instance.

      Currently I've worked around by implementing my own SessionFactory, which itself return custom Session instances supporting such listener.
      With a listener on standard/provided Session instance may not require user to to do such custom implementations.

      1. slistener.patch
        5 kB
        Cédric Chantepie

        Issue Links

          Activity

          Transition Time In Source Status Execution Times Last Executer Last Execution Date
          Open Open Resolved Resolved
          111d 16h 34m 1 Guillaume Nodet 01/Jul/11 08:50
          Guillaume Nodet made changes -
          Status Open [ 1 ] Resolved [ 5 ]
          Assignee Guillaume Nodet [ gnt ]
          Fix Version/s 0.6.0 [ 12315514 ]
          Fix Version/s 0.5.0 [ 12314994 ]
          Resolution Fixed [ 1 ]
          Hide
          Guillaume Nodet added a comment -

          Thx for the patch Cédric.

          Show
          Guillaume Nodet added a comment - Thx for the patch Cédric.
          Cédric Chantepie made changes -
          Link This issue is blocked by SSHD-108 [ SSHD-108 ]
          Hide
          Cédric Chantepie added a comment -

          In current patch, listeners are also added at SessionFactory level. As far as I understand it could match your expectation.
          As for listeners at Session level, I have thought that some people would prefer to "monitor" not all session but just some of them.
          It could be removed if no one think it could be useful.

          Show
          Cédric Chantepie added a comment - In current patch, listeners are also added at SessionFactory level. As far as I understand it could match your expectation. As for listeners at Session level, I have thought that some people would prefer to "monitor" not all session but just some of them. It could be removed if no one think it could be useful.
          Hide
          Guillaume Nodet added a comment -

          Hi Cédric,
          While reviewing this patch, it occured to me that it may be better to add the listeners for sessions at a upper level so that the listeners could be informed when sessions are created. I think that would be more symetric and would allow a single listener to be registered instead of having to register it for each session.

          Show
          Guillaume Nodet added a comment - Hi Cédric, While reviewing this patch, it occured to me that it may be better to add the listeners for sessions at a upper level so that the listeners could be informed when sessions are created. I think that would be more symetric and would allow a single listener to be registered instead of having to register it for each session.
          Cédric Chantepie made changes -
          Field Original Value New Value
          Attachment slistener.patch [ 12473478 ]
          Hide
          Cédric Chantepie added a comment -

          Quick patch

          Show
          Cédric Chantepie added a comment - Quick patch
          Hide
          Cédric Chantepie added a comment -

          I can try

          Show
          Cédric Chantepie added a comment - I can try
          Hide
          Bernd Fondermann added a comment -

          Cédric, do you want to contribute a patch?

          Show
          Bernd Fondermann added a comment - Cédric, do you want to contribute a patch?
          Cédric Chantepie created issue -

            People

            • Assignee:
              Guillaume Nodet
              Reporter:
              Cédric Chantepie
            • Votes:
              0 Vote for this issue
              Watchers:
              1 Start watching this issue

              Dates

              • Created:
                Updated:
                Resolved:

                Time Tracking

                Estimated:
                Original Estimate - 48h
                48h
                Remaining:
                Remaining Estimate - 48h
                48h
                Logged:
                Time Spent - Not Specified
                Not Specified

                  Development