Project

General

Profile

Defect #1412

Identity contract expiration lrt throws Exception

Added by Patrik Stloukal 7 months ago. Updated 7 months ago.

Status:
Closed
Priority:
Normal
Category:
Roles
Target version:
Start date:
12/11/2018
Due date:
% Done:

100%


Description

I reproduced this locally on new DB with IdM v 9.3.0
I made this steps:

- I create new user.
- and I create 2 roles - main, subordinate
- Then I create 'business role' - main as superior role and subordinate as subordinate role
- I change name of position of contract (that I can found it easier in DB)
- I add both roles to user
- user has 3 roles: main, subordinate and subordinate (the second through business role)
- in pgAdmin I set to user's contract valid till in past
- then I manually run IdentityContractExpirationTaskExecutor (to disable this user)
- IdentityContractExpirationTaskExecutor ends with EmptyResultDataAccessException

if IdentityContractExpirationTaskExecutor is started again it ends sucessfully (for this item), but with each EmptyResultDataAccessException lrt is stopped.

History

#1 Updated by Radek Tomiška 7 months ago

  • Category changed from Scheduler to Roles
  • Target version set to Opal (9.4.0-rc.1)

#2 Updated by Radek Tomiška 7 months ago

  • Status changed from New to In Progress
  • Target version changed from Opal (9.4.0-rc.1) to Onyx (9.3.2)

#3 Updated by Radek Tomiška 7 months ago

  • Status changed from In Progress to Needs feedback
  • Assignee changed from Radek Tomiška to Ondřej Kopr
  • % Done changed from 0 to 90

I fixed long running tasks:
- IdentityRoleExpirationTaskExecutor
- IdentityContractExpirationTaskExecutor

Now business roles are removed correctly on the first attempt. I've added integration tests.

Commit:
https://github.com/bcvsolutions/CzechIdMng/commit/407ef53a34ae1ba46011f5b578982b18f4a605fe

Could you do a feedback, please?

#4 Updated by Ondřej Kopr 7 months ago

  • Status changed from Needs feedback to Resolved
  • Assignee changed from Ondřej Kopr to Radek Tomiška
  • % Done changed from 90 to 100

Thanks for report and fix the issue. I tested exactly same behavior as is described in ticket and behavior with removing doesn't throw error. Nice.

#5 Updated by Radek Tomiška 7 months ago

  • Status changed from Resolved to Closed

Also available in: Atom PDF