@mikem

I really do not like that this complex behavior was added to the previously very simple getSessionInfo function. It is a consistency problem.

That said, adding these fields is fine for now.

Commit a40ce211 fixes this pull-request

Looks like I merged this a bit too early. Did we ever get qe testing on it?

I'm looking into it and it seems there's still a difference in the calls. getSessionInfo() without any args returns extra fields: renew_time and renew_ts, which aren't present when called with specific user_id.

  Full diff:
    [
     'authtype',
     'callnum',
     'exclusive',
     'expired',
     'master',
  +  'renew_time',
  +  'renew_ts',
     'start_time',
     'start_ts',
     'update_time',
     'update_ts',
     'user_id',
    ]

However, if that's fine, fields start_ts and update_ts were added correctly

Ah, yes, that should probably be added

https://pagure.io/koji/pull-request/3927