Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

sig#803 - Deletion of technical user during connector deletion #967

Open
MaximilianHauer opened this issue Sep 3, 2024 · 2 comments
Open

sig#803 - Deletion of technical user during connector deletion #967

MaximilianHauer opened this issue Sep 3, 2024 · 2 comments
Assignees
@Phil91
Labels
bug Something isn't working
Milestone
Release 2.2.0

Comments

@MaximilianHauer
Copy link

MaximilianHauer commented Sep 3, 2024 •
edited by Phil91
Loading

Current Behavior

If i delete a connector i can currently select the option to delete the responding technical user.
Currently the technical user is only set to inactive but this is wrong.

Expected Behavior

If i approve to delete the connector which is linked to an external user then execute the identical logic as we have in place for the technical user deletion endpoint.

Steps To Reproduce
@MaximilianHauer MaximilianHauer added the bug Something isn't working label Sep 3, 2024
@MaximilianHauer MaximilianHauer added this to the Release 2.2.0 milestone Sep 3, 2024
@Phil91 Phil91 self-assigned this Sep 3, 2024
Phil91 added a commit that referenced this issue Sep 3, 2024
@Phil91
feat(connector): adjust connector deletion
1492479 
* add flag to define whether a linked service account should be deleted
* adjust deletion logic for service accounts when deleting a connector

Refs: #966 #967
@Phil91 Phil91 mentioned this issue Sep 3, 2024
Merged
8 tasks
Phil91 added a commit that referenced this issue Sep 6, 2024
@Phil91
feat(connector): adjust connector deletion
4da7b52 
* add flag to define whether a linked service account should be deleted
* adjust deletion logic for service accounts when deleting a connector

Refs: #966 #967
Phil91 added a commit that referenced this issue Sep 6, 2024
@Phil91
feat(connector): adjust connector deletion
a593dbe 
* add flag to define whether a linked service account should be deleted
* adjust deletion logic for service accounts when deleting a connector

Refs: #966 #967
@MaximilianHauer MaximilianHauer mentioned this issue Sep 6, 2024
Open
14 tasks
Phil91 added a commit that referenced this issue Sep 9, 2024
@Phil91
feat(connector): adjust connector deletion
a3ec89d 
* add flag to define whether a linked service account should be deleted
* adjust deletion logic for service accounts when deleting a connector

Refs: #966 #967
Phil91 added a commit that referenced this issue Sep 9, 2024
@Phil91
feat(connector): adjust connector deletion
adb7ca9 
* add flag to define whether a linked service account should be deleted
* adjust deletion logic for service accounts when deleting a connector

Refs: #966 #967
Phil91 added a commit that referenced this issue Sep 10, 2024
@Phil91
feat(connector): adjust connector deletion
66c05e4 
* add flag to define whether a linked service account should be deleted
* adjust deletion logic for service accounts when deleting a connector

Refs: #966 #967
Phil91 added a commit that referenced this issue Sep 10, 2024
@Phil91
feat(connector): adjust connector deletion (#968)
c84034c 
* add flag to define whether a linked service account should be deleted
* adjust deletion logic for service accounts when deleting a connector
------------------------
Refs: #966 #967
Reviewed-By: Evelyn Gurschler <[email protected]>
@MaximilianHauer
Copy link
Author

@Phil91 i did create a tec user id 282fff21-741a-411a-ae69-cf79e95cb991 attached it to a connector - waited until the connector is active- deleted the connector but the tec user stays active.

@Phil91
Copy link
Member

Phil91 commented Sep 11, 2024

@Phil91 i did create a tec user id 282fff21-741a-411a-ae69-cf79e95cb991 attached it to a connector - waited until the connector is active- deleted the connector but the tec user stays active.

@MaximilianHauer did the call to the backend has the flag deleteTechnicalUser set to true? I'd say it was missing or set to false and therefor it didn't work.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@Phil91 Phil91

Labels
bug Something isn't working
Projects
Portal
Status: IN REVIEW
Milestone
Release 2.2.0
Development

No branches or pull requests
2 participants
@Phil91 @MaximilianHauer