2016-02-16 16:13 GMT+01:00 Richard W.M. Jones <rjones@redhat.com>:
On Mon, Feb 15, 2016 at 11:22:27PM +0100, Matthias Bolte wrote:
2016-02-11 12:59 GMT+01:00 Richard W.M. Jones <rjones@redhat.com>:
Is calling SessionIsActive necessary? From my (very limited) understanding, it seems as if we might use 'SessionManager. currentSession' property instead, which doesn't require admin permissions. Actually the code [see link above] already does this when ctx->hasSessionIsActive is false, but that doesn't apply to modern vCenter.
SessionIsActive Is not necessary here, but it seemed to be the better solution compared to this more hacky way of checking session object. I wasn't aware that is would require admin level privileges.
The patch works fine for me. I believe the server I'm testing against is vCenter 5.5, although I'm not totally sure about that.
ACK.
Rich.
Thanks, pushed. -- Matthias Bolte http://photron.blogspot.com