Fixes: https://pagure.io/koji/issue/3396
rebased onto 95b56b3f574402cdf6f3e16a9d33c23794b7a0c5
Just name it consistently with logged_user_id as user_id.
logged_user_id
user_id
and -> or (otherwise admin will not be able to get info about other users)
Why it should be filtered on start_time in this case?
start_time
It is a dict or list of dicts.
This can be retrieved directly from query date_part('epoch', start_time)
date_part('epoch', start_time)
It is auser which kojiweb uses to auth. Get proxied user via environ['koji.currentUser'] (E.g. when I'm not logged in in koji-dev env, it will still return kojiadmin user)
environ['koji.currentUser']
AND expired IS TRUE AND id = %(session_id)s - it is returning 0/1 records. (Or alternatively use EXISTS for bool check) We should also rewrite it to QueryProcessor when we're here (same for next query).
AND expired IS TRUE AND id = %(session_id)s
QueryProcessor
rebased onto ed89b8725d30b90d74c6e48d3e3129b4220b32b4
rebased onto 12f173b7553fd76688225940dcf29e8eee755dd7
rebased onto 01e68ddafe7094ae559495696af06bf043ef8566
@tkopecek fixed
rebased onto 6bf759953f0843538f414cf74ecfb320a7a1924e
It should be a default clause - we don't care about expired sessions, they should be handled by GC not this call.
rebased onto a8bbeef2f452c518a1c7413677f905f650fac26b
:thumbsup:
Metadata Update from @tkopecek: - Pull-request tagged with: testing-ready
rebased onto 5984b612a460455f8f6fcfafaf450151e585ef32
rebased onto 07c325e47e4427778959be17e44fc2f15470f0a4
rebased onto 8ca74441610e3461427cf766c6c1dd4365dc3b3d
Metadata Update from @jobrauer: - Pull-request untagged with: testing-ready
testing-ready removed due to merge conflict on our test branch
rebased onto 17d3e730ab5b0220863b3a513e7eea58c7dc2e86
rebased onto 151bf7414601e7acf945259603e685ad42aa80e3
rebased onto 87b16370498007b6c7e151eb68d561b2f97a512a
rebased onto 2eee0c87206b9df2399b54e5c6aee8fbac254de9
Metadata Update from @jcupova: - Pull-request tagged with: testing-ready
rebased onto 86d928203efe0a7d55ac6607a5b4f2d769953e21
Metadata Update from @jobrauer: - Pull-request tagged with: testing-done
Commit 08a9fb88 fixes this pull-request
Pull-Request has been merged by tkopecek
Fixes: https://pagure.io/koji/issue/3396