Project

General

Profile

Actions

Defect #1412

closed

Identity contract expiration lrt throws Exception

Added by Patrik Stloukal over 5 years ago. Updated over 5 years ago.

Status:
Closed
Priority:
Normal
Assignee:
Radek Tomiška
Category:
Roles
Target version:
Start date:
12/11/2018
Due date:
% Done:

100%

Estimated time:
1.00 h
Affected versions:
Owner:

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.

Actions #1

Updated by Radek Tomiška over 5 years ago

  • Category changed from Scheduler to Roles
  • Target version set to Opal (9.4.0-rc.1)
  • Estimated time set to 1.00 h
Actions #2

Updated by Radek Tomiška over 5 years ago

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

Updated by Radek Tomiška over 5 years 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?

Actions #4

Updated by Ondřej Kopr over 5 years 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.

Actions #5

Updated by Radek Tomiška over 5 years ago

  • Status changed from Resolved to Closed
Actions

Also available in: Atom PDF