Skip to content

Commit

Permalink
feat: modify delete user secret logic for using system token
Browse files Browse the repository at this point in the history
  • Loading branch information
ImMin5 committed Oct 25, 2024
1 parent d6b1684 commit 00030fa
Show file tree
Hide file tree
Showing 2 changed files with 7 additions and 2 deletions.
2 changes: 1 addition & 1 deletion src/spaceone/secret/manager/user_secret_manager.py
Original file line number Diff line number Diff line change
Expand Up @@ -39,7 +39,7 @@ def _rollback(old_data):
def delete_user_secret_by_vo(user_secret_vo):
user_secret_vo.delete()

def get_user_secret(self, user_secret_id, domain_id, user_id=None):
def get_user_secret(self, user_secret_id: str, domain_id: str, user_id: str = None):
conditions = {
"user_secret_id": user_secret_id,
"domain_id": domain_id,
Expand Down
7 changes: 6 additions & 1 deletion src/spaceone/secret/service/user_secret_service.py
Original file line number Diff line number Diff line change
Expand Up @@ -113,9 +113,14 @@ def delete(self, params):
"""

user_secret_id = params["user_secret_id"]
user_id = params["user_id"]
domain_id = params["domain_id"]

role_type = self.transaction.get_meta("role_type")
if role_type == "SYSTEM_TOKEN":
user_id = None
else:
user_id = params["user_id"]

user_secret_vo = self.user_secret_mgr.get_user_secret(
user_secret_id, domain_id, user_id
)
Expand Down

0 comments on commit 00030fa

Please sign in to comment.