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 - Enable deletion of a connectior without deleting the tec user #966

Open
3 tasks
MaximilianHauer opened this issue Sep 3, 2024 · 0 comments
Open
3 tasks

sig#803 - Enable deletion of a connectior without deleting the tec user #966

MaximilianHauer opened this issue Sep 3, 2024 · 0 comments
Assignees
@Phil91
Labels
enhancement New feature or request
Milestone
Release 2.2.0

Comments

@MaximilianHauer
Copy link

MaximilianHauer commented Sep 3, 2024 •
edited by Phil91
Loading

Description

If I do not select the deletion of the technical user then only the connection between connector and technical user should be removed. Therefor an additional parameter is needed for the endpoint DELETE: /api/administration/connectors/{connectorId}

Acceptance Criteria

  • Deleting a connector without the "delete user flag" being active should result in the deletion of the connection between the connector and the TEC user only. The connector itself should not be deleted in this case.
  • enhance endpoint DELETE: /api/administration/connectors/{connectorId}with query parameter deleteServiceAccount
  • provide unit tests
@MaximilianHauer MaximilianHauer added the enhancement New feature or request 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
chore: add file header
3722dfa 
Refs: #966
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]>
@ntruchsess ntruchsess mentioned this issue Sep 10, 2024
Merged
4 tasks
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@Phil91 Phil91

Labels
enhancement New feature or request
Projects
Portal
Status: IN REVIEW
Milestone
Release 2.2.0
Development

No branches or pull requests
2 participants
@Phil91 @MaximilianHauer