refactor(manage_security): simplify delete_swagger_user function
This commit is contained in:
parent
8a22e285df
commit
5f40c677a8
1 changed files with 7 additions and 9 deletions
|
|
@ -123,8 +123,7 @@ async def list_swagger_users():
|
|||
logger.info("")
|
||||
|
||||
|
||||
async def delete_swagger_user(username: str, tags: Optional[List[str]] = None):
|
||||
"""Supprimer un utilisateur Swagger"""
|
||||
async def delete_swagger_user(username: str):
|
||||
async with async_session_factory() as session:
|
||||
result = await session.execute(
|
||||
select(SwaggerUser).where(SwaggerUser.username == username)
|
||||
|
|
@ -132,12 +131,12 @@ async def delete_swagger_user(username: str, tags: Optional[List[str]] = None):
|
|||
user = result.scalar_one_or_none()
|
||||
|
||||
if not user:
|
||||
logger.error(f" Utilisateur '{username}' introuvable")
|
||||
logger.error(f"❌ Utilisateur '{username}' introuvable")
|
||||
return
|
||||
|
||||
await session.delete(user)
|
||||
await session.commit()
|
||||
logger.info("🗑️ Utilisateur Swagger supprimé: {}".format(username))
|
||||
logger.info(f"🗑️ Utilisateur Swagger supprimé: {username}")
|
||||
|
||||
|
||||
async def create_api_key(
|
||||
|
|
@ -306,8 +305,8 @@ Exemples:
|
|||
add_p.add_argument("--full-name", help="Nom complet", default=None)
|
||||
add_p.add_argument(
|
||||
"--tags",
|
||||
nargs="+", # Au moins 1 tag requis SI spécifié
|
||||
help="Tags OpenAPI autorisés (ex: Clients Devis). Vide = admin complet",
|
||||
nargs="*",
|
||||
help="Tags autorisés (Clients Devis etc). Vide ou omis = admin complet",
|
||||
default=None,
|
||||
)
|
||||
|
||||
|
|
@ -342,13 +341,12 @@ Exemples:
|
|||
|
||||
if args.command == "swagger":
|
||||
if args.swagger_command == "add":
|
||||
tags = args.tags if args.tags is not None else None
|
||||
tags = args.tags if args.tags else None
|
||||
await add_swagger_user(args.username, args.password, args.full_name, tags)
|
||||
elif args.swagger_command == "list":
|
||||
await list_swagger_users()
|
||||
elif args.swagger_command == "delete":
|
||||
tags = args.tags if args.tags is not None else None
|
||||
await delete_swagger_user(args.username, tags)
|
||||
await delete_swagger_user(args.username)
|
||||
else:
|
||||
swagger_parser.print_help()
|
||||
|
||||
|
|
|
|||
Loading…
Reference in a new issue