Project

General

Profile

Actions
Copy link

Task #3424

closed

Change UID during process virtual system request failing

Added by Ondřej Kopr over 1 year ago. Updated about 1 year ago.

Status:
Closed
Priority:
Normal
Assignee:
Michal Steinhart
Category:
Virtual systems
Target version:
12.2.7
Start date:
08/08/2023
Due date:
% Done:

100%

Estimated time:
Owner:
Peter Štrunc

Description

Requests after changing UID for virtual system failing for opened request that will be processed after the UID change.

Goal of the ticket is create behavior that allows solve requests after UID change.

Actions
Copy link
 #2

Updated by Ondřej Kopr over 1 year ago

  • Status changed from New to In Progress
Actions
Copy link
 #3

Updated by Ondřej Kopr over 1 year ago

  • % Done changed from 0 to 10

A probable solution will be create new processor with checking open requests and change UID for all these requests.

Actions
Copy link
 #4

Updated by Ondřej Kopr over 1 year ago

Warning message for future search:

2023-08-09 16:11:47.449  WARN 214309 --- [http-nio-8080-exec-7] eu.bcvsolutions.idm.core.exception.ExceptionControllerAdvice.log : [vs:VS_REQUEST_UPDATING_ACCOUNT_NOT_EXIST:722ae55f-479a-4b01-91dc-c03e6044a821] Virtual system account with UID [one] was not found. You have to relized 'create' request first! ({uid=one})
eu.bcvsolutions.idm.vs.exception.VsException: Virtual system account with UID [one] was not found. You have to relized 'create' request first!
    at eu.bcvsolutions.idm.vs.service.impl.DefaultVsRequestService.internalExecute(DefaultVsRequestService.java:355)
    at eu.bcvsolutions.idm.vs.service.impl.DefaultVsRequestService.realize(DefaultVsRequestService.java:171)
    at eu.bcvsolutions.idm.vs.service.impl.DefaultVsRequestService$$FastClassBySpringCGLIB$$21439a19.invoke(<generated>)
    at org.springframework.cglib.proxy.MethodProxy.invoke(MethodProxy.java:218)
    at org.springframework.aop.framework.CglibAopProxy$CglibMethodInvocation.invokeJoinpoint(CglibAopProxy.java:752)
Actions
Copy link
 #5

Updated by Ondřej Kopr over 1 year ago

  • Status changed from In Progress to Needs feedback
  • Assignee changed from Ondřej Kopr to Michal Steinhart
  • % Done changed from 10 to 90

Finally was request implemented.

Whole processors implementation was removed and changed with service implementations. The behavior is more clear than processors.

Commit: https://github.com/bcvsolutions/CzechIdMng/commit/a23e695e2265fa67942b39dde62700c0b0ea55ed (branch: hotfix-12.2.7)

Please Michal could you check/test the new behavior?

Thank you.

Actions
Copy link
 #6

Updated by Michal Steinhart about 1 year ago

  • Owner changed from Ondřej Kopr to Peter Štrunc

Finally tested.
The new behaviour works well.
The change of username(UID) creates new request for VS administrator after resolving this request can be resolved the origin request with also.

Works even though if a username (UID) is changed more than once.

It is ready to merge, please.

Actions
Copy link
 #7

Updated by Peter Štrunc about 1 year ago

  • Status changed from Needs feedback to Closed
  • % Done changed from 90 to 100

Merged released

Actions
Copy link

Also available in: Atom PDF