Ensuring a 401 error is returned when user data is not found in multi-user context.

This commit is contained in:
Laurent Erignoux 2025-03-30 10:13:13 +08:00
parent 61c8c70c6e
commit 7ff5fc84af

View File

@ -9,8 +9,14 @@ class AppSettings():
self.user_manager = user_manager self.user_manager = user_manager
def get_settings(self, request): def get_settings(self, request):
try:
file = self.user_manager.get_request_user_filepath( file = self.user_manager.get_request_user_filepath(
request, "comfy.settings.json") request,
"comfy.settings.json"
)
except KeyError as e:
log.error(f"User settings not found.")
raise aiohttp.web.HTTPUnauthorized('User not found') from e
if os.path.isfile(file): if os.path.isfile(file):
try: try:
with open(file) as f: with open(file) as f:
@ -30,8 +36,10 @@ class AppSettings():
def add_routes(self, routes): def add_routes(self, routes):
@routes.get("/settings") @routes.get("/settings")
async def get_settings(request): async def get_settings(request):
return web.json_response(self.get_settings(request)) return web.json_response(self.get_settings(request))
@routes.get("/settings/{id}") @routes.get("/settings/{id}")
async def get_setting(request): async def get_setting(request):
value = None value = None